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.

Sign up for GitHub

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

Jump to bottom

Deduplicate RequestInfo params. #1329

Merged
merged 1 commit into from
Sep 17, 2022
Merged

Deduplicate RequestInfo params. #1329

merged 1 commit into from
Sep 17, 2022

Conversation

Ku33ma
Copy link

@Ku33ma Ku33ma commented Jul 29, 2020

This is a more correct way to solve the problem described in #1325.
Any duplicate fields are removed.

@TravisBuddy
Copy link

Hey @Ku33ma,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: 55371bf0-d1cc-11ea-bb2f-e1a2ee6530fc

@Ku33ma Ku33ma changed the title Deduplicate RequestInfo params. #1325 Deduplicate RequestInfo params. Jul 29, 2020
@jpegxguy jpegxguy mentioned this pull request Aug 5, 2020
Open
This was referenced Sep 9, 2020
Open
Open
@PeterDaveHello PeterDaveHello requested a review from a team September 29, 2020 06:56
@anna328p anna328p mentioned this pull request Nov 19, 2020
Closed
@ambroisie
Copy link

Is there any news regarding this MR?

@PeterDaveHello
Copy link
Member

Waiting for review from @transmission-remote-gui/development

@PeterDaveHello PeterDaveHello merged commit 1c81df7 into transmission-remote-gui:master Sep 17, 2022
@PeterDaveHello
Copy link
Member

Let's see if it works.

@ambroisie
Copy link

@PeterDaveHello I've been running it with this patch at least since my comment, so I can confirm that it works correctly.

@PeterDaveHello
Copy link
Member

@ambroisie thanks!

@elbarbudo
Copy link

Same thing for me : after upgrading my mediacenter, transmission-daemon become version 3.0 and transgui V5.18.0 does not run anymore.
Transmission daemon from OSMC for Raspberry pi
transgui on Linux Mint 21

BUT transgui v5.18.0 on Raspberry PI400 runs fine

@ambroisie
Copy link

@elbarbudo your raspberry pi's distribution probably applied this patch downstream, whereas Linux Mint hasn't.

AndDT pushed a commit to AndDT/SlackBuildsOrg that referenced this pull request Mar 31, 2023
@bassmadrigal @aclemons
network/transgui: Fix duplicate object status error.
8cac518 
Fix "Duplicate object member: status" error

Thanks to Ku33ma

transmission-remote-gui/transgui#1329

Signed-off-by: Andrew Clemons <andrew.clemons@gmail.com>
willysr pushed a commit to SlackBuildsOrg/slackbuilds that referenced this pull request Apr 1, 2023
@bassmadrigal @willysr
network/transgui: Fix duplicate object status error.
aec811a 
Fix "Duplicate object member: status" error

Thanks to Ku33ma

transmission-remote-gui/transgui#1329

Signed-off-by: Andrew Clemons <andrew.clemons@gmail.com>

Signed-off-by: Willy Sudiarto Raharjo <willysr@slackbuilds.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@Ku33ma @TravisBuddy @ambroisie @PeterDaveHello @elbarbudo