Update awork extension#28337
Conversation
- Merge branch \'fix-memory-issue\' into \'develop\' - Add changelog - Add token update via watcher - Fix memeory issue - Pull contributions
|
Thank you for your contribution! 🎉 🔔 @Malte-Hypercode 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. 📋 Quick checkout commandsBRANCH="ext/awork"
FORK_URL="https://github.com/Malte-Hypercode/raycast-extensions.git"
EXTENSION_NAME="awork"
REPO_NAME="raycast-extensions"
git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run devWe're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days. |
Greptile SummaryThis PR replaces the previous self-revalidating
Confidence Score: 5/5The logic changes are well-scoped and fix a real revalidation-loop problem; the only issue is a cosmetic changelog date that should use the standard placeholder. All functional changes are straightforward and improve existing behaviour. The token-change subscription is correctly cleaned up in useEffect, the retry delay is properly applied before re-issuing rate-limited requests, and the normalization helpers guard against missing nested fields. The only finding is the hardcoded date in CHANGELOG.md. No files require special attention beyond swapping the hardcoded date in CHANGELOG.md for {PR_MERGE_DATE}. Important Files Changed
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
extensions/awork/CHANGELOG.md:3
The new changelog entry uses a hardcoded date instead of the `{PR_MERGE_DATE}` placeholder required by the Raycast contribution guidelines. The merge date is automatically populated when the PR is merged — hardcoding today's date will leave the wrong release date in the changelog if the PR merges on a different day.
```suggestion
## [Improve stability] - {PR_MERGE_DATE}
```
Reviews (3): Last reviewed commit: "Update CHANGELOG.md" | Re-trigger Greptile |
…tatus codes - Added a delayBeforeRetry function to introduce a 1-second wait before retrying requests that fail with a 429 status. - Updated catch blocks in getProjects, getTasks, and getTypesOfWork to utilize the new delay function for improved error handling.
pernielsentikaer
left a comment
There was a problem hiding this comment.
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. |
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder