Labels Milestones

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

Merged

jose merged 7 commits from onboard-alert-component into master 3 weeks ago

Conversation 2 Commits 7 Files Changed 5

jose commented 4 weeks ago

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 4 weeks ago

7f5a64dceb UI: update HomeView onboard alert style

- Style-wise, make it consistent with similar alerts
- Introduce visual hierarchy to buttons
- Responsive button layout

47a7b03cca Refactor: simplify RegistrationNotice component interface

- 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.

2cb2f3ef3a Fix: update notice locator

25e79249dd Feat: integrate RegistrationNotice component into ProjectsView

- 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 3 weeks ago

27b46b4252 feat: Reorder and reword the 'meeting' registration option based on feedback.

trentlarson commented 3 weeks ago

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 3 weeks ago

7b357641d1 Chore: update author in RegistrationNotice

jose commented 3 weeks ago

Poster

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 3 weeks ago

ecadb00396 Merge branch 'master' into onboard-alert-component

jose merged commit 580a485573 into master 3 weeks ago

jose referenced this issue from a commit 3 weeks ago

Merge pull request 'Refactor: create reusable component version of registration/onboarding notice' (#179) from onboard-alert-component into master

The pull request has been merged as 580a485573.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

No items

No Label

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.