Update tmdb extension#9508
Conversation
- Update Changelog - Add \'Copy TMDB ID\' to Detail Pages - Feature #9502 add ability to copy title id from search - Initial commit
|
Thank you for your contribution! 🎉 🔔 @Aayush9029 @andreaselia @thomaslombart you might want to have a look. |
|
Looks good to me, but do we really need to display the ID we are copying? Could the action simply be "Copy TMDB ID"? Just thinking if it's useful to show the ID here or better to keep it simple 🤔 |
|
LGTM imo, It’s fine to display the ID since the length of the string is small. |
|
@andreaselia I was thinking of that, but the ID isn't shown anywhere, hence why I would add it in the Action menu. That way the user knows what it copied. It's only a few numbers, so it wouldn't mess up the Action panel. If the ID was displayed in the Details, I would remove it from being displayed in the Action title I guess, does that make sense? Let me know if I should remove it from the title :) |
|
Let me know if we should keep the movie ID in the action or not. I'd be in favor of having it as just "Copy TMDb ID" since it's cleaner and seeing the ID doesn't provide much value, but that's just my two cents 🙂 |
- Linting - Remove TMDB ID from action title, add ID to detail
|
@andreaselia @thomaslombart I updated the PR, I removed the ID from the Action title, added it to the Details. See screenshots: edit: This is only added to the details screen for Search, not the other detail pages |
|
The looks good, nice work dude - I'm happy for this to be merged @thomaslombart 👍 |
|
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 (@j3lte). |



Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder