Add Merge Strategy Option to Swagger API #1952

Closed
opened 2025-11-02 04:18:57 -06:00 by GiteaMirror · 6 comments
Owner

Originally created by @mohammed90 on GitHub (Jun 19, 2018).

  • Gitea version (or commit ref): N/A
  • Git version: 3f2f575
  • Operating system: N/A
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
    N/A

Description

Currently the UI allows for multiple PR merge strategies controlled by the form value "do". The funcs allow for this control, but the Swgger API doesn't expose the option.

The SDK is also missing the feature as seen here. Should I create another issue for the SDK in its corresponding repo?

Screenshots

N/A

Originally created by @mohammed90 on GitHub (Jun 19, 2018). - Gitea version (or commit ref): N/A - Git version: 3f2f575 - Operating system: N/A - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - [ ] Not relevant - Log gist: N/A ## Description Currently the UI allows for multiple PR merge strategies controlled by the form value "do". The funcs allow for this control, but the Swgger API doesn't expose the option. The SDK is also missing the feature as seen [here](https://godoc.org/code.gitea.io/sdk/gitea#Client.MergePullRequest). Should I create another issue for the SDK in its corresponding repo? ## Screenshots N/A
GiteaMirror added the issue/confirmedtype/bug labels 2025-11-02 04:18:57 -06:00
Author
Owner

@stale[bot] commented on GitHub (Jan 23, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Jan 23, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@mohammed90 commented on GitHub (Jan 23, 2019):

/keep

@mohammed90 commented on GitHub (Jan 23, 2019): /keep
Author
Owner

@6543 commented on GitHub (Dec 31, 2019):

@Mohammed90 https://gitea.com/api/swagger#/repository/repoMergePullRequest

-> API now has a do field wich is interpreted as Enum: [ merge, rebase, rebase-merge, squash ]

@6543 commented on GitHub (Dec 31, 2019): @Mohammed90 https://gitea.com/api/swagger#/repository/repoMergePullRequest -> API now has a `do` field wich is interpreted as Enum: [ merge, rebase, rebase-merge, squash ]
Author
Owner

@6543 commented on GitHub (Dec 31, 2019):

@Mohammed90 if you are fine with - can you close this issue?

@6543 commented on GitHub (Dec 31, 2019): @Mohammed90 if you are fine with - can you close this issue?
Author
Owner

@6543 commented on GitHub (Dec 31, 2019):

PS: for the SDK there is a PR https://gitea.com/gitea/go-sdk/pulls/207

@6543 commented on GitHub (Dec 31, 2019): PS: for the SDK there is a PR https://gitea.com/gitea/go-sdk/pulls/207
Author
Owner

@mohammed90 commented on GitHub (Dec 31, 2019):

That PR is not merged yet. I'll close it when it's merged.

@mohammed90 commented on GitHub (Dec 31, 2019): That PR is not merged yet. I'll close it when it's merged.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#1952