Skip to content

Update transmission extension#11838

Merged
raycastbot merged 3 commits intoraycast:mainfrom
j3lte:ext/transmission
Apr 22, 2024
Merged

Update transmission extension#11838
raycastbot merged 3 commits intoraycast:mainfrom
j3lte:ext/transmission

Conversation

@j3lte
Copy link
Contributor

@j3lte j3lte commented Apr 17, 2024 •
edited
Loading

Description

  • Refactor the code, splitting the listTorrents file into separate components
  • Update command because Cmd + , is reserved by Raycast
  • Fix a bug where the extension tries to read files that are not in localhost
  • Fix a memory leak in TorrentConfiiguration component
  • Fix a bug where too many files are loaded in a big torrent (added a configurable limit (max 100))
  • Update dependencies, cleaning up unnecessary ones

Screencast

Checklist

@j3lte
Update transmission extension
d2aa59b 
- Update changelog
- Updates
- Update dependencies, cleanup package.json
- Update cmd + ,
- Refactor code
- Initial commit
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: transmission Issues related to the transmission extension labels Apr 17, 2024
@raycastbot
Copy link
Collaborator

raycastbot commented Apr 17, 2024 •
edited
Loading

Thank you for your contribution! 🎉

🔔 @FezVrasta @danpalmer you might want to have a look.

pernielsentikaer
pernielsentikaer reviewed Apr 19, 2024
View reviewed changes
@pernielsentikaer
Copy link
Collaborator

pernielsentikaer commented Apr 19, 2024

Thanks for the update @j3lte 🔥

@FezVrasta do you mind checking this 🙂

@pernielsentikaer pernielsentikaer self-assigned this Apr 19, 2024
pernielsentikaer
pernielsentikaer approved these changes Apr 22, 2024
View reviewed changes
Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a 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 🔥

@raycastbot raycastbot merged commit c1e97c4 into raycast:main Apr 22, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Apr 22, 2024

Published to the Raycast Store:
https://raycast.com/FezVrasta/transmission

@raycastbot
Copy link
Collaborator

raycastbot commented Apr 22, 2024

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

@klizas
Copy link

klizas commented Apr 22, 2024

Thank you so much! 🙇

@j3lte j3lte deleted the ext/transmission branch April 22, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pernielsentikaer pernielsentikaer pernielsentikaer approved these changes

+1 more reviewer

@FezVrasta FezVrasta FezVrasta approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@pernielsentikaer pernielsentikaer

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: transmission Issues related to the transmission extension

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Transmission] Runs out of memory loading a large list of torrents

5 participants

@j3lte @raycastbot @pernielsentikaer @klizas @FezVrasta