Refactor: create reusable component version of registration/onboarding notice #179

Merged
jose merged 7 commits from onboard-alert-component into master 2025-08-25 06:53:37 +00:00
Owner
  • Improve existing notice style by introducing visual hierarchy to buttons and laying them out responsively
  • Centralize all registration notice logic within upgraded RegistrationNotice component, for use in HomeView, AccountViewView and ProjectsView
  • Clean up unused methods and imports in affected views
  • Enable customizable messages to maintain original contextual wordings unique to each view
  • Update Playwright test 00 to use the correct ID locator
- Improve existing notice style by introducing visual hierarchy to buttons and laying them out responsively - Centralize all registration notice logic within upgraded RegistrationNotice component, for use in HomeView, AccountViewView and ProjectsView - Clean up unused methods and imports in affected views - Enable customizable messages to maintain original contextual wordings unique to each view - Update Playwright test 00 to use the correct ID locator
jose added 4 commits 2025-08-21 11:40:04 +00:00
- Style-wise, make it consistent with similar alerts
- Introduce visual hierarchy to buttons
- Responsive button layout
- Centralize all registration notice logic within upgraded RegistrationNotice component
- Clean up unused methods and imports in HomeView and AccountViewView
- Customizable message to maintain original contextual wordings unique to each view

Component is now more focused and follows Vue.js best practices
for conditional rendering.
- Replace custom onboarding button with RegistrationNotice component
- Maintain v-else conditional rendering for unregistered users
- Remove unused methods and imports related to custom registration flow
- Use contextual message "To announce a project, get someone to onboard you."
- Remove unnecessary wrapping div in RegistrationNotice

Completes registration UI standardization across all main views.
trentlarson added 1 commit 2025-08-23 01:56:32 +00:00
Owner

Looks great! 👍

Note that your commit included a reference to Matthew @anomalist

Looks great! 👍 Note that your commit included a reference to Matthew @anomalist
jose added 1 commit 2025-08-25 06:51:45 +00:00
Author
Owner

Looks great! 👍

Note that your commit included a reference to Matthew @anomalist

Updated the author before merging.

> Looks great! 👍 > > Note that your commit included a reference to Matthew @anomalist Updated the author before merging.
jose added 1 commit 2025-08-25 06:53:20 +00:00
jose merged commit 580a485573 into master 2025-08-25 06:53:37 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#179