Skip to content
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 Changelog #20991

Draft
wants to merge 1 commit into
base: v5_0_x
Choose a base branch
from

Conversation

sledgehammer999
Copy link
Member

Please review the changelog entries for v5 RC1.
I have left the commit hash to ease review.

Also I have taken into account the stuff from PR #20918

PS: This is just for review. After it is approved I'll push it manually, deleting the hashes.

@sledgehammer999 sledgehammer999 added this to the 5.0 milestone Jun 25, 2024
- BUGFIX: Display error message when unrecoverable error occurred (glassez)
- BUGFIX: Update size of selected files when selection is changed (glassez)
- BUGFIX: Normalize tags by trimming leading/trailing whitespace (glassez)
- BUGFIX: Correctly handle share limits in torrent options dialog (glassez)
- BUGFIX: Adjust tracker tier when adding additional trackers (Chocobo1) 321d7e5b17b6e4cacf713b4f4409364a5e65873e
- BUGFIX: Fix inconsistent naming between (Done/Progress) column (luzpaz) d3315f7cc7f31b1578cf72f07b175a80c204dac9
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- BUGFIX: Fix inconsistent naming between (Done/Progress) column (luzpaz) d3315f7cc7f31b1578cf72f07b175a80c204dac9
- BUGFIX: Fix inconsistent naming between Done/Progress column (luzpaz) d3315f7cc7f31b1578cf72f07b175a80c204dac9

- WEBAPI: Fix wrong timestamp values (Chocobo1)
- WEBAPI: Send binary data with filename and mime type specified (glassez)
- WEBAPI: Expose API for the torrent creator (glassez, Radu Carpa)
- WEBAPI: API for listing directory content (Paweł Kotiuk) a1af077889de587d8c71968a380abc9eb4aea47f
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- WEBAPI: API for listing directory content (Paweł Kotiuk) a1af077889de587d8c71968a380abc9eb4aea47f
- WEBAPI: Provide endpoint for listing directory content (Paweł Kotiuk) a1af077889de587d8c71968a380abc9eb4aea47f

- WEBAPI: Fix wrong timestamp values (Chocobo1)
- WEBAPI: Send binary data with filename and mime type specified (glassez)
- WEBAPI: Expose API for the torrent creator (glassez, Radu Carpa)
- WEBAPI: API for listing directory content (Paweł Kotiuk) a1af077889de587d8c71968a380abc9eb4aea47f
- WEBAPI: Provide "isPrivate" flag via "torrents/info" endpoint (ManiMatter) 05416458db79e050f8500e1d543148ce4cf2ecdb
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- WEBAPI: Provide "isPrivate" flag via "torrents/info" endpoint (ManiMatter) 05416458db79e050f8500e1d543148ce4cf2ecdb
- WEBAPI: Provide "private" flag via "torrents/info" endpoint (ManiMatter) 05416458db79e050f8500e1d543148ce4cf2ecdb

- WEBAPI: API for listing directory content (Paweł Kotiuk) a1af077889de587d8c71968a380abc9eb4aea47f
- WEBAPI: Provide "isPrivate" flag via "torrents/info" endpoint (ManiMatter) 05416458db79e050f8500e1d543148ce4cf2ecdb
- WEBAPI: Add a way to download .torrent file using search plugin (glassez) 4d8713ce110d80f995212fb7a779fe16504fb91d
- WEBAPI: Add "private" filter for 'info' endpoint (ManiMatter) 914728d9a102ee8aa44eb46c413ecb668728709c
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- WEBAPI: Add "private" filter for 'info' endpoint (ManiMatter) 914728d9a102ee8aa44eb46c413ecb668728709c
- WEBAPI: Add "private" filter for 'torrents/info' endpoint (ManiMatter) 914728d9a102ee8aa44eb46c413ecb668728709c

@glassez
Copy link
Member

glassez commented Jun 25, 2024

c36100f should be mentioned as WINDOWS: Don't access "Favorites" folder unexpectedly.

- FEATURE: Focus on Download button if torrent link retrieved from the clipboard (glassez) 7310eec74e6694342e9934ecc524df38d0064bb5
- FEATURE: Add ability to pause/resume entire BitTorrent session (glassez) 8ef7d3ec9a56e7445948c37a282629ca5ed551ab
- FEATURE: Add an option to set BitTorrent session shutdown timeout (glassez) 2b728b3bc0c525905ef4c6086bc45027eb2150fd
- FEATURE: Apply filename filter to subfolder names as well (glassez) d89f289f82fb6ff4305841c7d4c7899ffe12f896
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- FEATURE: Apply filename filter to subfolder names as well (glassez) d89f289f82fb6ff4305841c7d4c7899ffe12f896
- FEATURE: Apply "Excluded file names" to folder names as well (glassez) d89f289f82fb6ff4305841c7d4c7899ffe12f896

I thought the current description was referring to "Filter files..." in content tab.

- BUGFIX: Fix inconsistent naming between (Done/Progress) column (luzpaz) d3315f7cc7f31b1578cf72f07b175a80c204dac9
- BUGFIX: Sanitize peer client names (Hanabishi) 2c47f09d7a797c1ea886879749b71198e449e701
- BUGFIX: Apply share limits when torrent downloading is finished (glassez) 65d143d4c411e045b3cd1cf847dee831290af799
- BUGFIX: Revise Protocol column (Chocobo1) 1c49e0973c94b32bf3bfe52845b6b9572c997ef2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No protocol column in 4.6.x so mentioning this is meaningless.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's reasonable.
But it is also reasonable to mention fixes from the previous public release of v5.0.x, which is v5.0beta1.
So I would mention all of them in the Changelog, both v5.0beta1 and v5.0RC1, as separate entries, and not just single v5.0. A lot of software do this and it looks good to me.
@sledgehammer999?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am thinking of promoting this to a FEATURE entry. "Add 'Protocol' column in trackers tab".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am thinking of promoting this to a FEATURE entry. "Add 'Protocol' column in trackers tab".

It isn't a feature in itself. It is just a side effect of another changes.

@Pentaphon

This comment was marked as off-topic.

@glassez

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants