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

plugins.phonebook_backend: fixed PhonebookSource definition #192

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DrPyser
Copy link
Contributor

@DrPyser DrPyser commented Jan 31, 2024

phonebook openapi schema PhonebookSource definition has response-only attributes that must be set to readOnly, in accordance with actual marshmallow schemas.

@DrPyser DrPyser added mergeit and removed mergeit labels Jan 31, 2024
Copy link
Contributor

Build succeeded.
https://zuul.wazo.community/zuul/t/local/buildset/d57c776f0a4b4cec85523baf2026b3e8

✔️ tox-linters SUCCESS in 7m 46s
✔️ wazo-tox-py39 SUCCESS in 7m 45s
✔️ debian-packaging-bullseye SUCCESS in 2m 23s
✔️ dird-tox-integration SUCCESS in 33m 57s

@fblackburn1 fblackburn1 force-pushed the fix-openapi-phonebook branch from f4b7f5b to dd8061c Compare February 1, 2024 12:47
@fblackburn1
Copy link
Member

(force push to stop zuul gate - will be merged after freeze)

Copy link
Contributor

Build succeeded.
https://zuul.wazo.community/zuul/t/local/buildset/bf58499ef43f4d06b4760f4cb7fda24b

✔️ tox-linters SUCCESS in 7m 52s
✔️ wazo-tox-py39 SUCCESS in 7m 49s
✔️ debian-packaging-bullseye SUCCESS in 2m 14s
✔️ dird-tox-integration SUCCESS in 33m 51s

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