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

UI/UX Changes #630

Merged
merged 29 commits into from
Sep 18, 2024
Merged

UI/UX Changes #630

merged 29 commits into from
Sep 18, 2024

Conversation

RonenSivak
Copy link
Contributor

This pull request includes updates to the overall UI and UX.
Some of the changes include:

  • A info toast that pops up if the client-id is the default one - telling the user to change the default client-id to his own client-id
  • Environment check to see if user is in Codesandbox platform and display a modal when user tries to login inside this environment
  • Some UI changes that include small changes to the examples display, and examples page display.
  • Some UX changes like added loaders when fetching items and blocking subscription example if user is not logged in
  • Stores product are now stacking when choosing the same item instead of duplicating
  • In the examples section, we are now checking what applications are installed and navigate to install page when clicked if application is not installed.
  • If user is not premium then pop up a premium modal when trying to check out

@RonenSivak
move client id out
7424190
@RonenSivak
added indications to user about he's apps
151e46b
@RonenSivak
changed back default client-id
04d1bb1
@RonenSivak
display the no app message only if theres no apps installed
60cb99e
@RonenSivak
added examples files
fa2f235
@RonenSivak
implement a new get msid method
1d21f4b
@RonenSivak
implement a new get msid method
a8e4bfd
@RonenSivak
detect client-id change so cookies will re-load to the current client-id
ec63183
@RonenSivak
restore default client-id
fa06bf4
@RonenSivak
design changes & prettier lint
37dccb6
@RonenSivak
changed layout to not show all apps on all pages.
9513e45 
changed booking design
@RonenSivak
changed store page design. cleaned the code a little-bit
eb27051
@RonenSivak
changed to events & tickets + lint
5a40e7c
@RonenSivak
some bug fixes, added modal to store in check out if user is not prem…
319af47 
…ium. added an error page to display and error in case client id is invalid
@RonenSivak
lint
3deb316
@RonenSivak
minor changes
5ec89d5
@RonenSivak
fixed cors error when trying to validate users redirect uri
12f3515
@RonenSivak
added global loader for async functions to give user indication that …
bc95397 
…something is loading
@RonenSivak
changed selected cards border opacity
3c75282
@RonenSivak
no need for uri validation
f8cc282
@RonenSivak
lint
58bad13
@RonenSivak
added comments
dcb1094
@RonenSivak
removed validations for url
826b5ab
@RonenSivak
tests fix + lint
ee19f1a
@RonenSivak
meta side fixes, warnings fixed for css, added loader for content
1a97b1a
@RonenSivak
seperated external and internal logic to ease understanding for users…
64e7ea4 
…. also added modal when trying to log in from codesandbox with the default client id
@vercel Vercel
Copy link

vercel bot commented Sep 5, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
wix-headless-example ✅ Ready (Inspect) Visit Preview Sep 18, 2024 7:49am

@netlify Netlify
Copy link

netlify bot commented Sep 5, 2024 •
edited
Loading

✅ Deploy Preview for wix-headless-example ready!

Name Link
🔨 Latest commit c2cb6d3
🔍 Latest deploy log https://app.netlify.com/sites/wix-headless-example/deploys/66ea85a62d3e3c00082a4995
😎 Deploy Preview https://deploy-preview-630--wix-headless-example.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RonenSivak
Merge branch 'refs/heads/main' into update-apps-display
592f8f8 
# Conflicts:
#	pages/booking.js
#	pages/login-callback.js
#	pages/members.js
#	pages/store.js
#	pages/subscriptions.js
#	pages/tickets.js
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 5, 2024

🟢 Playwright E2E for NETLIFY

Name Info
Summary Action Summary
Site URL https://deploy-preview-630--wix-headless-example.netlify.app/

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 5, 2024

🟢 Playwright E2E for NETLIFY

Name Info
Summary Action Summary
Site URL https://deploy-preview-630--wix-headless-example.netlify.app/

liorMar
liorMar approved these changes Sep 18, 2024
View reviewed changes
Copy link
Contributor

@liorMar liorMar left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@github-actions GitHub Actions
Copy link
Contributor

🟢 Playwright E2E for NETLIFY

Name Info
Summary Action Summary
Site URL https://deploy-preview-630--wix-headless-example.netlify.app/

@RonenSivak RonenSivak merged commit b5335b9 into main Sep 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liorMar liorMar liorMar 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
@RonenSivak @liorMar