Update jwt-decoder extension#9442
Conversation
- fix jwt-view not showing - Initial commit
|
Thank you for your first contribution! 🎉 🔔 @gdsmith you might want to have a look. |
|
Thanks for your contribution @zhhanging 🔥 Do you mind adding a CHANGELOG entry too? @gdsmith can you approve this? |
- Merge branch \'contributions/merge-1701151083113667000\' - Pull contributions - add CHANGELOG.md
|
I have Added a CHANGELOG.md file as suggested. |
|
@gdsmith are you around? |
gdsmith
left a comment
There was a problem hiding this comment.
@zhhanging thanks so much for fixing this, I mostly use the list view so hadn't noticed this had broken. Any chance you can update the other encodeURI use as per my comment?
| const dataItems = ListFromObject(data, claims); | ||
|
|
||
| const tokenImg = `<img alt="view token" width="720" src="data:image/svg+xml,${encodeURI( | ||
| const tokenImg = `<img alt="view token" width="720" src="data:image/svg+xml,${encodeURIComponent( |
There was a problem hiding this comment.
There's another use of encodeURI in jwt-list.tsx can you replace that one too?
There was a problem hiding this comment.
I have replaced the encodeURI in jwt-list.tsx as suggested. Thank you.
pernielsentikaer
left a comment
There was a problem hiding this comment.
Hi 👋
Looks good to me, approved 🔥
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 Such a great contribution deserves a reward, but unfortunately we couldn't find your Raycast account based on your GitHub username (@zhhanging). |
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder