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

feat UGN-328 UGN-331 - add hooks to each step #87

Merged
merged 2 commits into from
Apr 20, 2022
Merged

feat UGN-328 UGN-331 - add hooks to each step #87

merged 2 commits into from
Apr 20, 2022

Conversation

JulitorK
Copy link
Contributor

  • added hook to each step
  • all step hooks are async - allows network calls
  • added loading states to indicate async actions

example:
Screenshot 2022-04-11 at 19 10 15

@JulitorK JulitorK requested a review from masiulis April 11, 2022 16:16
masiulis
masiulis reviewed Apr 13, 2022
View reviewed changes
src/steps/UploadFlow.tsx Outdated Show resolved Hide resolved
masiulis
masiulis approved these changes Apr 13, 2022
View reviewed changes
Copy link
Contributor

@masiulis masiulis left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@JulitorK JulitorK merged commit 68551f3 into master Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@masiulis masiulis masiulis 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.
2 participants
@JulitorK @masiulis