Skip to content

Conversation

@Sander0542
Copy link
Contributor

Currently, the API schema has some missing properties. This PR adds the missing one so that we have a more complete API schema.

Closes #4356

Related to Sander0542/terraform-provider-nginxproxymanager#240

@jc21
Copy link
Member

jc21 commented Dec 1, 2025

Latest CI Error:

$.paths['/users'].post.responses['201'].content['application/json'].examples['default']
additional properties 'modified_on', 'id', 'created_on', 'user_id' not allowed

This linting is done with vacuum using the rules defined here.

@Sander0542
Copy link
Contributor Author

Thanks for letting me know! I'll try to push a fix later today

@nginxproxymanagerci
Copy link

Docker Image for build 8 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-4998

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

@Sander0542
Copy link
Contributor Author

Latest CI Error:

$.paths['/users'].post.responses['201'].content['application/json'].examples['default']
additional properties 'modified_on', 'id', 'created_on', 'user_id' not allowed

This linting is done with vacuum using the rules defined here.

@jc21 I fixed the issue and the pipeline just succeeded :)

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.

2 participants