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

Add the commit toggle field the merge endpoint POST serializer #24

Closed
mrmrcoleman opened this issue Aug 8, 2024 · 1 comment
Closed
Assignees
Labels
type: bug A confirmed report of unexpected behavior in the application

Comments

@mrmrcoleman
Copy link

Plugin Version

v0.2.0 (aa7f933)

NetBox Version

v4.1-beta1

Python Version

3.10

Steps to Reproduce

  • Create a branch
  • View the swagger schema and form the merge POST request based on what is shown there
  • Attempt to merge via POST request

Expected Behavior

Following the Swagger schema I expect to be able to form a valid merge POST request

Observed Behavior

The merge POST request throws a 500 KeyError because it requires the commit field toggle

@mrmrcoleman mrmrcoleman added the type: bug A confirmed report of unexpected behavior in the application label Aug 8, 2024
Copy link

linear bot commented Aug 8, 2024

BRA-31 Add the commit toggle field the merge endpoint POST serializer

Plugin Version

v0.2.0 (aa7f933)

NetBox Version

v4.1-beta1

Python Version

3.10

Steps to Reproduce

  • Create a branch
  • View the swagger schema and form the merge POST request based on what is shown there
  • Attempt to merge via POST request

Expected Behavior

Following the Swagger schema I expect to be able to form a valid merge POST request

Observed Behavior

The merge POST request throws a 500 KeyError because it requires the commit field toggle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

No branches or pull requests

2 participants