-
Notifications
You must be signed in to change notification settings - Fork 274
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
Dynamic dialogs based on selections #185
Dynamic dialogs based on selections #185
Conversation
@ousszizou is attempting to deploy a commit to the Salgsmaskin Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@alexghirelli - Going to merge this now. Can you handle the input to the DB? |
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 good! Merging now! ✨
Great Work, but could we also add these features @ousszizou ? Is it possible to adjust the number formatting to include a space between each set of three digits (thousand separators)? This change would enhance readability for users, helping them quickly grasp the value presented. Could we transform the number format from the current representation? To a more user-friendly format with spaces (link to the second image)? Additionally, for consistency and improved user experience, can we add a "Name" input field at the top of all related interfaces ✨ Tasks
|
Yeaa for sure. I'll work on that today. |
Thanks! If you want to raise a new PR you can do that :) |
Definitely! that's what Am going to do. |
Description
What type of PR is this? (check all applicable)
Related Tickets & Documents
Close #166
Mobile & Desktop Screenshots/Recordings
Steps to QA
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?