Skip to content

Comments

Improvements to Setup Flow#196

Open
DKeatonDC wants to merge 3 commits intomainfrom
feature/setupFlowImprovements
Open

Improvements to Setup Flow#196
DKeatonDC wants to merge 3 commits intomainfrom
feature/setupFlowImprovements

Conversation

@DKeatonDC
Copy link
Contributor

Critical Changes

Adds usability to setup flow that were initially part of PR #189 and #147

Changes

Several changes to the setup flow to address multiple Issues: flow usability and design updates, fonts, button labels, flow element sizing consistent for display in Lightning Page; added welcome screens and instructions depending on first time or re-entry ; added help text on Company Name to describe its use ; next button language is now consistent, this was already resolved )

Issues Closed

#93
#102
#144
#147
#177

Adds improvements to setup flow that were initially part of PR #189
Copy link
Contributor

@acrosman acrosman left a comment

Choose a reason for hiding this comment

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

@DKeatonDC I'm not sure that having the flow description include changes makes sense in this context. If the changes are important we should document them in the project somehow – like the readme or a wiki page – but some of the purpose of putting those comments in flows is the lack of good change tracking on platform. Git solves that problem for us. Otherwise the changes look good to me.

@DKeatonDC
Copy link
Contributor Author

@DKeatonDC I'm not sure that having the flow description include changes makes sense in this context. If the changes are important we should document them in the project somehow – like the readme or a wiki page – but some of the purpose of putting those comments in flows is the lack of good change tracking on platform. Git solves that problem for us. Otherwise the changes look good to me.

Hi @acrosman I improve the flow description. Feel free to approve.

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