Hi @blestab, could I take this?

Yes you may @aguptamusic , thanks

@blestab @jackcmeyer
The design provided above for the "+ New Note" button doesn't match the style of the "+ Add New Note" button in the patient notes view. The "+ New Note" design above has a green background and looks similar to the "Resolve Incident" button, whereas the "+ Add New Note" has a white background that turns blue when hovering. Which should we use?

Thanks @aguptamusic please go ahead and use button label, functionality (display message if there are no notes) and design/styling that matches the Patient Notes view. In addition though, we need a table of notes for this one instead of how Patient Notes displays the list of notes, with the ability to delete/edit a note.

@blestab Given that the core team seems to still be working out logging in functionality, it is not obvious what role the current user plays (Doctor, etc.). How should we work with the givenBy field in a Note? A few options we're thinking about:

1. Try to replicate App.tsx and use the current session and user object from a try/catch and handle the inevitable error case from RemoteDB.
2. Comment out the givenBy field and ignore it for now.
3. Insert a dummy string in the field, making them all uniform for now.

What do you recommend we do?

You can get the current user from the redux store, which currently has mock data inside of it. You can see an example of how to do that in the report incident functionality.

If a user updates a note with the "Edit" functionality, should the date/givenBy fields be updated?

If a user updates a note with the "Edit" functionality, should the date/givenBy fields be updated?

I don't believe so as those are related to the "create" event. In future we could possibly look at introducing lastEditDate, and lastEditBy fields to track edits.

Makes sense.

The patient module also has a notes section. Would you like us to also make the notes section for the patient module use the same Notes Table UI? Currently the patient notes tab is a list without support for edit/delete.

Makes sense.

The patient module also has a notes section. Would you like us to also make the notes section for the patient module use the same Notes Table UI? Currently the patient notes tab is a list without support for edit/delete.

Good idea. That would be great for UX consistency; We will log a separate issue for it.