Skip to content

feat: implement notifications system and UI#155

Merged
owengretzinger merged 1 commit intomainfrom
01-12-feat_notification_page
Jan 27, 2025
Merged

feat: implement notifications system and UI#155
owengretzinger merged 1 commit intomainfrom
01-12-feat_notification_page

Conversation

@owengretzinger
Copy link
Collaborator

@owengretzinger owengretzinger commented Jan 17, 2025

TL;DR

Added notifications screen with support for friend requests, habit invites, and nudges.

Screen.Recording.2025-01-17.at.1.01.42.PM.mov

What changed?

  • Added notifications screen with real-time updates and interactive responses
  • Upgraded @gorhom/bottom-sheet from 4.6.3 to 5.0.6 to gain access to dynamic sizing: https://gorhom.dev/react-native-bottom-sheet/dynamic-sizing
  • Created new components for notification cards and user info displays
  • Implemented mock notification data and handlers
  • Added single habit query hook

How to test?

  1. Navigate to the notifications tab
  2. Verify different notification types display correctly:
    • Friend requests
    • Habit invites
    • Nudge reminders
  3. Test notification interactions:
    • Respond to friend requests
    • Accept/decline habit invites
    • Dismiss nudges
  4. Confirm user profiles and habit details load in notification modals
  5. Verify time ago formatting works correctly

@owengretzinger owengretzinger changed the title feat: notification page feat: implement notifications system and UI Jan 17, 2025
@owengretzinger owengretzinger force-pushed the 01-12-feat_notification_page branch from 0a3bbef to a0f664b Compare January 17, 2025 18:04
@owengretzinger owengretzinger marked this pull request as ready for review January 17, 2025 18:06
@graphite-app graphite-app bot requested a review from AnkushSarkar10 January 17, 2025 18:06
@graphite-app
Copy link

graphite-app bot commented Jan 17, 2025

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (01/17/25)

1 reviewer was added to this PR based on Owen Gretzinger's automation.

@owengretzinger owengretzinger force-pushed the 01-11-feat_habit_info_page branch from 55a6a59 to 0bf032d Compare January 27, 2025 07:46
@owengretzinger owengretzinger force-pushed the 01-12-feat_notification_page branch from a0f664b to 76b6087 Compare January 27, 2025 07:47
@owengretzinger owengretzinger force-pushed the 01-11-feat_habit_info_page branch 2 times, most recently from 02845cd to c57cbc9 Compare January 27, 2025 07:54
@owengretzinger owengretzinger force-pushed the 01-12-feat_notification_page branch from 76b6087 to 3aa2f64 Compare January 27, 2025 07:54
@owengretzinger owengretzinger changed the base branch from 01-11-feat_habit_info_page to graphite-base/155 January 27, 2025 07:59
@owengretzinger owengretzinger force-pushed the 01-12-feat_notification_page branch from 3aa2f64 to 4d8ea59 Compare January 27, 2025 08:00
@owengretzinger owengretzinger changed the base branch from graphite-base/155 to main January 27, 2025 08:00
@owengretzinger owengretzinger force-pushed the 01-12-feat_notification_page branch from 4d8ea59 to 3e713eb Compare January 27, 2025 08:00
@owengretzinger owengretzinger merged commit 9097a7a into main Jan 27, 2025
2 checks passed
Copy link
Collaborator Author

Merge activity

  • Jan 27, 3:03 AM EST: A user merged this pull request with Graphite.

@owengretzinger owengretzinger deleted the 01-12-feat_notification_page branch January 27, 2025 08:04
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