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

[WBX-103] - FE1 to FE2 Messaging #2121

Merged
merged 43 commits into from
Mar 14, 2024

Conversation

andrewwallacespeckle
Copy link
Contributor

@andrewwallacespeckle andrewwallacespeckle commented Mar 13, 2024

Description & motivation

As FE2 becomes the default, we need to guide FE1 users towards the new web app. This ticket covers the changes to both FE1 and FE2.

Changes:

  • Added FE1 Dialog with video and messaging directing user to FE2.
  • FE1 Dialog is controlled by LocalStorage, which is set when the user skips the dialog
  • Added a new block to the left hand menu in FE1, directs to FE2 and Blog Post
  • Added FE2 Banner welcoming users to FE2. CTA launches dialog
  • Added Dialog to FE2, controlled by Cookie
  • Added Menu item to revert to FE1
  • Small design changes to segmentation, so it is more fitting with FE2.
  • Add messaging to FE1 Login page

To-do before merge:

  • Ensure connectors team are ready
  • Ensure growth team are ready
  • Ensure web team are ready
  • Ensure Decision Makers are aware and happy to go live

Screenshots:

image image image image image image image

@andrewwallacespeckle andrewwallacespeckle marked this pull request as ready for review March 13, 2024 13:20
Copy link
Contributor

@fabis94 fabis94 left a comment

Choose a reason for hiding this comment

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

left some comments

Copy link
Contributor

@fabis94 fabis94 left a comment

Choose a reason for hiding this comment

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

added a couple of comments, but theres also a pre-commit issue with helm chart changes that I don't know how to resolve 🤔

@fabis94 fabis94 marked this pull request as ready for review March 14, 2024 12:44
@andrewwallacespeckle andrewwallacespeckle changed the title [WBX-103] - FE1 to FE2 Messaging - DO NOT MERGE YET [WBX-103] - FE1 to FE2 Messaging Mar 14, 2024
@andrewwallacespeckle andrewwallacespeckle merged commit 3c83d31 into main Mar 14, 2024
31 of 34 checks passed
@andrewwallacespeckle andrewwallacespeckle deleted the andrew/WBX-103-FE1-FE2-Messaging branch March 14, 2024 13:19
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.

3 participants