Skip to content

New missing token user message#88

Merged
guillaq merged 1 commit intomainfrom
guillaume/new-missing-token-user-message
Apr 10, 2025
Merged

New missing token user message#88
guillaq merged 1 commit intomainfrom
guillaume/new-missing-token-user-message

Conversation

@guillaq
Copy link
Collaborator

@guillaq guillaq commented Apr 10, 2025

@guillaq guillaq requested review from pierrevalade and yannbu April 10, 2025 09:44
Copy link
Contributor

@yannbu yannbu left a comment

Choose a reason for hiding this comment

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

Nice ! Thanks

@guillaq guillaq force-pushed the guillaume/new-missing-token-user-message branch from 2e68a06 to 27ee41e Compare April 10, 2025 12:09
@guillaq guillaq merged commit 70ae66a into main Apr 10, 2025
5 checks passed
@guillaq guillaq deleted the guillaume/new-missing-token-user-message branch April 10, 2025 12:11
Copy link
Contributor

@guillaq what's the UX when there isn't a try/catch ? i can test myself too when the new SDK version is out.

Copy link
Contributor

Screenshot 2025-04-11 at 7.48.24 AM.png

just checked without the try/catch — looks great too! thank you @guillaq

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.

3 participants