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

Redesign Home Interaction #555

Merged
merged 26 commits into from
Jan 11, 2024
Merged

Redesign Home Interaction #555

merged 26 commits into from
Jan 11, 2024

Conversation

garrettmflynn
Copy link
Member

This PR implements recommendations made by @oruebel (in user test) and @rly (#531) to improve the user experience of interacting with the GUIDE home screen.

In particular, it swaps the original home screen with the Conversions page so that users can quickly engage with the main feature of the GUIDE: designing conversion pipelines.

@garrettmflynn garrettmflynn self-assigned this Jan 8, 2024
@CodyCBakerPhD
Copy link
Collaborator

Attempting to run a fresh tutorial pipeline on this PR results in my backend server crashing and restarting before it can even launch the pipeline; do I need to do something special when testing this branch?

@garrettmflynn
Copy link
Member Author

Changing the conversion page path messed up the progress resume function. I've updated this now.

@CodyCBakerPhD
Copy link
Collaborator

Love the consistency of the new workflow names!

All works for me

@CodyCBakerPhD CodyCBakerPhD enabled auto-merge (squash) January 10, 2024 20:15
@CodyCBakerPhD
Copy link
Collaborator

Storybook has some problems that start on the first couple of pipeline pages though

@garrettmflynn
Copy link
Member Author

Good catch! Similar issue, just fixed

@CodyCBakerPhD CodyCBakerPhD self-requested a review January 10, 2024 21:06
@CodyCBakerPhD
Copy link
Collaborator

Good catch! Similar issue, just fixed

Most still seem to be empty: https://www.chromatic.com/test?appId=640a4d93e34604f275368983&id=659f074e4699c6ad0c249ece

@garrettmflynn
Copy link
Member Author

Got it. That one had an inconsistent format for the updated line so it was missed on my first pass. Thanks!

@CodyCBakerPhD CodyCBakerPhD self-requested a review January 11, 2024 00:04
@CodyCBakerPhD CodyCBakerPhD merged commit 7367494 into main Jan 11, 2024
10 checks passed
@CodyCBakerPhD CodyCBakerPhD deleted the home-redesign branch January 11, 2024 00:09
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