-
Notifications
You must be signed in to change notification settings - Fork 4.8k
[Envato] Fix issue where "accountEnvato" would crash on first run due to missing cache + more #23072
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
Conversation
|
Thank you for your contribution! 🎉 🔔 @Slllava @3llomi you might want to have a look. You can use this guide to learn how to check out the Pull Request locally in order to test it. Due to our current reduced availability, the initial review may take up to 10 business days. |
Greptile OverviewGreptile SummaryThis PR modernizes the Envato extension with critical bug fixes and improved error handling. The key fix prevents Major Changes:
Confidence Score: 4/5
Important Files ChangedFile Analysis
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15 files reviewed, 2 comments
pernielsentikaer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, approved 🔥
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
NOTE
I have left some
anyin the extension for compatibility. I will attempt to remove in future updates with improved fetching process altogether.Description
Screencast
envato-account-crash.yafw.balanced.mp4
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare located outside the metadata folder if they were not generated with our metadata tool