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.

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

UX: Allows going back to dashboard when cancelling from there #527

Closed
wants to merge 22 commits into from
Closed

UX: Allows going back to dashboard when cancelling from there #527

wants to merge 22 commits into from

Conversation

zomars
Copy link
Member

@zomars zomars commented Aug 25, 2021

fixes #525

This PR also fixes linting and typing errors of the modified pages. The class names were sorted with the recommended tailwind extension that you have on the repo.

@vercel
Copy link

vercel bot commented Aug 25, 2021

@zomars is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

pages/bookings/index.tsx Outdated Show resolved Hide resolved
pages/bookings/index.tsx Outdated Show resolved Hide resolved
pages/cancel/[uid].tsx Outdated Show resolved Hide resolved
@zomars zomars requested a review from KATT August 26, 2021 18:00
Omar López added 12 commits August 26, 2021 12:29
# Conflicts:
#	pages/bookings/index.tsx
# Conflicts:
#	pages/cancel/[uid].tsx

# Conflicts:
#	pages/cancel/[uid].tsx
Also:
- Linting
- Type fixes

# Conflicts:
#	pages/cancel/success.tsx
# Conflicts:
#	pages/bookings/index.tsx
# Conflicts:
#	pages/cancel/[uid].tsx
…0525

# Conflicts:
#	pages/bookings/index.tsx
#	pages/cancel/[uid].tsx
#	pages/cancel/success.tsx
@zomars zomars changed the base branch from main to l10n_main August 27, 2021 19:38
@zomars zomars changed the base branch from l10n_main to main August 27, 2021 19:38
@zomars
Copy link
Member Author

zomars commented Aug 27, 2021

Rebased from main @KATT ready for re-review

@PeerRich
Copy link
Member

@zomars can you provide screenshots or better, a loom.com video?

pages/bookings/index.tsx Show resolved Hide resolved
pages/bookings/index.tsx Outdated Show resolved Hide resolved
pages/cancel/[uid].tsx Outdated Show resolved Hide resolved
pages/cancel/[uid].tsx Outdated Show resolved Hide resolved
pages/cancel/success.tsx Outdated Show resolved Hide resolved
pages/cancel/success.tsx Outdated Show resolved Hide resolved
pages/cancel/success.tsx Outdated Show resolved Hide resolved
Omar López added 3 commits September 15, 2021 08:45
# Conflicts:
#	pages/bookings/index.tsx
#	pages/cancel/[uid].tsx
#	pages/cancel/success.tsx
@zomars
Copy link
Member Author

zomars commented Sep 15, 2021

Since this functionality is already implemented, this PR just became minor type fixes @KATT Feel free to review, close or merge as you see fit.

@pumfleet pumfleet closed this Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Canceling from /bookings doesn't take you back to this page
5 participants