-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: Wire up design settings (button colour) #2669
Conversation
Removed vultr server and associated DNS entries |
6f06ae0
to
2f737c2
Compare
Trying this out on the pizza I noticed that the new form does update the DB but the change isn't seen in the flows. Is that part of the scope of this or is that work elsewhere / to be done later? |
a542870
to
64793b4
Compare
@Mike-Heneghan Sorry should have clarified that - this code change in this PR doesn't update the button colour directly, that was updated in this PR - #2658 I've just merged that and rebased this PR though so this should now work end-to-end 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great and working as expected for me on the pizza now 🥳
What does this PR do?