Skip to content

[feature/pinning-notes]#540

Merged
aritra-tech merged 10 commits intoaritra-tech:developfrom
andy-ife:pin-notes-feature
Nov 2, 2024
Merged

[feature/pinning-notes]#540
aritra-tech merged 10 commits intoaritra-tech:developfrom
andy-ife:pin-notes-feature

Conversation

@andy-ife
Copy link
Contributor

@andy-ife andy-ife commented Nov 1, 2024

Goal

This pull request adds the ability for users to pin and unpin important notes to the top of the notes list. Related issue here

Implementation Details

Key details:

  • The Note model now has an extra field: isPinned, which also adds an extra column to the database.
  • Database has been updated from v6 to v7.
  • Notes Screen now has a dynamic bottom sheet with pin and delete options.
  • Pin and unpin logic is as simple as note.isPinned = true/false, then update database.
  • Dao get methods now returns notes sorted by pinned status and dateTime.
  • Notes can also be pinned/unpinned in the AddEditScreen

Examples

See a short video of the feature here

Copy link
Owner

@aritra-tech aritra-tech left a comment

Choose a reason for hiding this comment

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

lgtm 👍🏻

@aritra-tech aritra-tech merged commit e687462 into aritra-tech:develop Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants