Skip to content

feat: react quickstart app style update - #40

Merged
fcoppede merged 3 commits into
mainfrom
react-update
Oct 13, 2025
Merged

feat: react quickstart app style update#40
fcoppede merged 3 commits into
mainfrom
react-update

Conversation

@fcoppede

Copy link
Copy Markdown
Contributor

updated the style, look and feel of the react quickstart app to match the idea we have for the SPA app:

  1. New font
  2. Match Zitadel colors for login/logout button
  3. Add decoded JWT to the home page
  4. Added more detailed instructions to the readme file
  5. Updated the oidc client to the latest version

@elinashoko elinashoko moved this to 📋 Sprint Backlog in Product Management Sep 30, 2025
Comment thread README.md
Comment thread README.md
Comment thread README.md Outdated
@mffap

mffap commented Oct 13, 2025

Copy link
Copy Markdown
Member

While trying out the example, I didn't follow the instructions exactly. So I ended up getting a bearer token, which didn't result in an error, just the start page again after a login.
I had to go to the console, but I also got only the information about a 400 status from the Zitadel response. Without any knowledge this would have been hard to debug.

Can we render error responses in the application?

@mffap
mffap self-requested a review October 13, 2025 11:23

@mffap mffap left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I left some comments, overall the application works. I just had a couple of comments that might improve the DevX a bit more.

Updated readme to make it more clear and fixed the error that happens when the access token is bearer (not jwt), so the app does not break and simply shows the access token without decoding it.
@fcoppede

Copy link
Copy Markdown
Contributor Author

thanks @mffap just updated the code based on your comments. I cleaned up the readme file and fixed the app so it will not break if the token is not in JWT format.

@fcoppede
fcoppede requested a review from mffap October 13, 2025 12:14
@fcoppede
fcoppede merged commit 6361b3a into main Oct 13, 2025
@github-project-automation github-project-automation Bot moved this from 📋 Sprint Backlog to ✅ Done in Product Management Oct 13, 2025
@fcoppede
fcoppede deleted the react-update branch October 13, 2025 14:40
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants