-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Simplified wizard step 3 and 5 accordingly #5334
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
Simplified wizard step 3 and 5 accordingly #5334
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/mglto8h9o |
Codecov Report
@@ Coverage Diff @@
## development #5334 +/- ##
============================================
Coverage 22.78% 22.78%
============================================
Files 491 491
Lines 5245 5245
Branches 37 37
============================================
Hits 1195 1195
Misses 4045 4045
Partials 5 5 Continue to review full report at Codecov.
|
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.
Thanks! Please resolve conflicts.
</div> | ||
</div> | ||
{{/each}} | ||
<br> |
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.
Please don't change indentation unnecessarily. I can't see what's your change and what's just indentation change. Revert to original indentation and just make your changes in the PR
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.
Sorry for that, actually i use Prettier and as soon as i saved the changes it formatted everything. I will make a new PR.
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.
No need for a new PR, push changes to this PR itself
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.
Unnecessary indentation changes
Fixes #5013
Short description of what this resolves:
Simplifies wizard Step 3 and 5
Changes proposed in this pull request:
Checklist
development
branch.