Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 96, 86, 85, 78 #120

Merged
merged 8 commits into from
Feb 19, 2025
Merged

Issue 96, 86, 85, 78 #120

merged 8 commits into from
Feb 19, 2025

Conversation

Daraujo26
Copy link
Contributor

[issue 96, 78] This code has the calendar pop up working and allows changes on the date selected inside create-journal-entry.

TODO:

  1. The database needs to be updated to be able to store this selected date.
  2. There may need to be minor UI adjustments on the calendar to match exactly whats on figma.

[issue 86, 85] I also went ahead and started reformatting the homepage to match whats on figma e.g. the profile and search button in the top right. I also fixed the add button on the bottom of the homepage. In the create page, I added the button in the bottom right as in figma so it shows a pop up for future actions to implement.

TODO:

  1. Add icons inside the actions popover in the create journal entry

Issue ticket number and link

#96
#86
#85
#78

Checklist

  • [yes] I have performed a self-review of my code
  • [yes] I have used npx expo-doctor to review any dependency issues.
  • [yes] I have pulled and merged any recent changes from the main branch (no merge conflicts)
  • [need database updated to store selected date entry] I have tested thoroughly any edge cases that might break the application
  • [yes] I have deleted any console.log statements and useless comments

@Avina20 Avina20 requested review from Avina20 and natzhu03 February 18, 2025 21:08
Copy link
Contributor

@Avina20 Avina20 left a comment

Choose a reason for hiding this comment

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

lgtm

@Avina20 Avina20 merged commit f079898 into main Feb 19, 2025
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.

4 participants