Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

fix(notes): add note details screen #165

Merged
marcin-kozinski merged 1 commit into from
Jan 31, 2025
Merged

fix(notes): add note details screen #165

marcin-kozinski merged 1 commit into from
Jan 31, 2025

Conversation

@marcin-kozinski
Copy link
Collaborator

@marcin-kozinski marcin-kozinski commented Jan 30, 2025 •
edited
Loading

View full note when clicking on a note row in Notes filter.

Screenshot

note details screen

References

PR Checklist

Setup:

  • Described changes for automated release notes in PR title using
    Conventional Commits standard
  • Self Review (review, clean up, documentation)
  • Basic Self QA
  • Feature flagged as needed to ensure this specific code is beta and production ready

Review:

  • Code Review approved

@marcin-kozinski marcin-kozinski added the notes Move to "Notes" section in release notes. label Jan 30, 2025
nzeltzer
nzeltzer approved these changes Jan 30, 2025
View reviewed changes
Copy link
Contributor

@nzeltzer nzeltzer left a comment

Choose a reason for hiding this comment

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

lgtm

@marcin-kozinski marcin-kozinski added this pull request to the merge queue Jan 31, 2025
Merged via the queue into main with commit 7d36a05 Jan 31, 2025
5 checks passed
@marcin-kozinski marcin-kozinski deleted the note-details branch January 31, 2025 08:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@nzeltzer nzeltzer nzeltzer approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

notes Move to "Notes" section in release notes.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@marcin-kozinski @nzeltzer