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.

Sign up for GitHub

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

Jump to bottom

Fix/general fixes #116

Merged
merged 4 commits into from
Oct 1, 2024
Merged

Fix/general fixes #116

merged 4 commits into from
Oct 1, 2024

Conversation

JosueBrenes
Copy link
Contributor

UI general fixes #109

Changes description

  • The input type for "Set your goal in STRKs" was changed to accept only numeric values.

  • The Discord link was updated to the correct one.

  • A button was added using an icon to return to phase 1 when in phase 2.

  • Alerts were implemented to notify when a text field in a form is left blank.

Current output

2024-09-30.23-31-21.mp4

image

image

image

2024-09-30.23-42-59.mp4

Time spent breakdown

  • Changing the Discord link: Approximately 5 minutes.
  • Implementing alerts: 1 hour.
  • Changing and validating the text field to number: 10 minutes.
  • Adding the button via the icon to return to phase 1: 1 hour.
  • Total time spent: Approximately 3 hours.

adrianvrj
adrianvrj approved these changes Oct 1, 2024
View reviewed changes
Copy link
Member

@adrianvrj adrianvrj left a comment

Choose a reason for hiding this comment

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

Great work!

@adrianvrj adrianvrj merged commit c1ec2b8 into web3wagers:dev Oct 1, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrianvrj adrianvrj adrianvrj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] UI general fixes
2 participants
@JosueBrenes @adrianvrj