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

Merging requests in the Web UI needs only Maintenance ID #1065

Closed
celan opened this issue Aug 27, 2015 · 7 comments
Closed

Merging requests in the Web UI needs only Maintenance ID #1065

celan opened this issue Aug 27, 2015 · 7 comments
Labels
Bug Frontend Things related to the OBS RoR app

Comments

@celan
Copy link

celan commented Aug 27, 2015

When trying to merge a maintenance request with an existing Maintenance incident it fails because it expects only the ID. In this case it needed '1035' and it rejected 'SUSE:Maintenance:1035'. The search field proposed the latter string. It doesn't find anything when providing only the ID.

@adrianschroeter
Copy link
Member

which interface are you using here? The webui?

@celan
Copy link
Author

celan commented Sep 10, 2015

Yes, that is in the Web UI.

@adrianschroeter adrianschroeter added Bug Frontend Things related to the OBS RoR app P3 labels Sep 10, 2015
@bgeuken
Copy link
Member

bgeuken commented Sep 24, 2015

SUSE:Maintenance:1035 is a project. The ui expects a request id of a request you want to merge it with.

@bbrunner
Copy link
Member

@bgeuken: No, it expects the ID of the maintenance-project. It's about "Merge with existing incidents" in requests with the type incident.

@bgeuken
Copy link
Member

bgeuken commented Sep 24, 2015

Ah, you are right the id parameter I saw comes from a hidden field tag:/

@bgeuken
Copy link
Member

bgeuken commented Sep 24, 2015

I think it's caused by this line. But the code is so complicated that I am really unsure about this. And there are no tests that would allow me to check this easily.

@hennevogel hennevogel removed the P3 label Jun 25, 2020
@hellcp-work
Copy link
Contributor

This functionality doesn't exist in the frontend anymore, see #8207

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Frontend Things related to the OBS RoR app
Projects
None yet
Development

No branches or pull requests

6 participants