Skip to content

feature: Add sorting options for notes.#129

Merged
aritra-tech merged 2 commits into
aritra-tech:developfrom
V9vek:feature/SortingOptionsForNotes
Oct 10, 2023
Merged

feature: Add sorting options for notes.#129
aritra-tech merged 2 commits into
aritra-tech:developfrom
V9vek:feature/SortingOptionsForNotes

Conversation

@V9vek

@V9vek V9vek commented Oct 8, 2023

Copy link
Copy Markdown
Contributor

Added the notes sorting feature, resolving the issue #118

It includes sorting notes based on:

Date (Oldest | Newest)
Title (A-Z | Z-A)
You can see the implementation in the attached video.
And one more thing, the Gradle Lint is showing error out of nowhere every time

If this looks good to you, please merge the request 😊
or if you need some changes, please do ping me.

noteapp.mp4

@aritra-tech

Copy link
Copy Markdown
Owner

Hey @V9vek can you please fix the ktlint check.
Just run ./gradlew ktlintCheck --continue in your terminal and check which lines are showing the error.

@V9vek

V9vek commented Oct 10, 2023

Copy link
Copy Markdown
Contributor Author

Hey @V9vek can you please fix the ktlint check. Just run ./gradlew ktlintCheck --continue in your terminal and check which lines are showing the error.

okay sure

@V9vek

V9vek commented Oct 10, 2023

Copy link
Copy Markdown
Contributor Author

@aritra-tech can you check now?

@aritra-tech aritra-tech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙌
Nice work @V9vek

@aritra-tech aritra-tech merged commit 8389997 into aritra-tech:develop Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants