-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Update hidemyemail extension #16015
Update hidemyemail extension #16015
Conversation
- improved user experience through caching and optimistic updates, ncreased timeout limit, added icons - Initial commit
Thank you for the update! 🎉 Due to our current reduced availability, the initial review may take up to 10-15 business days 🎄 |
- added information to list item metadata, fixed bugs - improved login / session expired handling
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.
- It feels a bit weird that I see the login screen everythime I open the command, do you have a good idea on how we can improve this?
Raycast02012025-VTiRD8gH.png.mp4
- linting fix - Fixed issue useCachedPromise() not fetching last cached state
I think I've found and fixed the issue although I'm not sure if it's me using hooks incorrectly or a bug when using both the Also, why is there a short moment of the "No Results" view even when rendering the MWE1:
MWE2:
|
I'll ask my collegaues about this next week since they're all on vacation until next week 🙂 In the latest change I keep getting logged out, I don't know if that's becaue of the latest change. In the video I just logged in and re-opened the command again. Raycast02012025-Wp0Z8Zlv.png.mp4 |
Haha, it seems you take working for a productivity tool very seriously ;) I'm not able to reproduce the issue. Did you try clearing local storage & cache? If this didn't resolve it, could you show me what is logged to the console when you open the command? |
I removed some of the URLs so you can see output
|
And what happens if you open the command, log in, and perform some action (e.g. deactivating an address). Does this work? If this does work, but on subsequent openings of the command it fails to log in, it seems to be an issue with loading/storing the cookies to local storage. Could you replace
and share the logs made to the console?
|
Here you go 🙂 I get prompted to insert the 2FA just fine
|
So it is saving and loading the data from local storage, but no cookies are actually stored after making the requests. Could you add the following line below line 76 in
so it becomes
I hope this solves the problem |
Still failed actually, said "logged in" but re-entering did the same
|
One where my proxy was disabled
|
Could you replace the function on line 93 in I'm sorry for the many requests, I'm new to such problems and trying to figure out how to debug it.
|
Can you ping me in Slack :) |
MWE1: Confirmed and will be in the next release MWE2: Confirmed, we're looking into a fix 🙂 |
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.
Hi 👋
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 build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are placed outside of themetadata
folder