[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #14 from Lucas Gass --- Doesn't apply cleanly in 22.05.x, if needed please rebase. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Blocks||33328 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33328 [Bug 33328] x-marc-schema should be renamed x-record-schema -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #13 from Matt Blenkinsop --- Nice work everyone! Pushed to stable for 22.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Matt Blenkinsop changed: What|Removed |Added Status|Pushed to master|Pushed to stable Version(s)|23.05.00|23.05.00,22.11.04 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Bug 33227 depends on bug 31800, which changed state. Bug 31800 Summary: REST API: POST endpoint for Biblios https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31800 What|Removed |Added Status|Pushed to stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #12 from Tomás Cohen Arazi --- Pushed to master for 23.05. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||23.05.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Martin Renvoize changed: What|Removed |Added Attachment #148410|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 148411 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148411=edit Bug 33227: (follow-up) Improve docs Signed-off-by: Tomas Cohen Arazi Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #10 from Tomás Cohen Arazi --- Created attachment 148410 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148410=edit Bug 33227: (follow-up) Improve docs Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Martin Renvoize changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #9 from Martin Renvoize --- This resolves the issues presented and causes now regressions. However, I'd love to see some documentation added somewhere to the API docs that states we expect MARC records to be passed (in various formats).. at the moment, the docs aren't all that clear. Anyways.. Passing QA to get us back on spec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Martin Renvoize changed: What|Removed |Added Attachment #148269|0 |1 is obsolete|| --- Comment #8 from Martin Renvoize --- Created attachment 148408 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148408=edit Bug 33227: Remove invalid spec and adjust the code accordingly The spec contains a non-standard definition for the body param. Removing it from the spec makes us need to handle the 'body' manually in the case of JSON data. This patch basically does that. It also changes the uses of $c->validation, which are discouraged by the Mojolicious::Plugin::OpenAPI dev/maintainer. I do it to highlight what we must do in other places and the fact that there's no behavior change. To test: 1. Apply this patch 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/biblios.t \ xt/api.t => SUCCESS: Tests pass! 3. Sign off :-D Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Martin Renvoize changed: What|Removed |Added Attachment #148268|0 |1 is obsolete|| --- Comment #7 from Martin Renvoize --- Created attachment 148407 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148407=edit Bug 33227: Unit tests This patch adds a unit test that launches the swagger-cli validator against our spec. Signed-off-by: Tomas Cohen Arazi Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Blocks||33250 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33250 [Bug 33250] DELETE /biblios/:biblio_id fails when ES is enabled -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Depends on||31800 See Also|https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=31800 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31800 [Bug 31800] REST API: POST endpoint for Biblios -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #6 from Tomás Cohen Arazi --- Created attachment 148269 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148269=edit Bug 33227: Remove invalid spec and adjust the code accordingly The spec contains a non-standard definition for the body param. Removing it from the spec makes us need to handle the 'body' manually in the case of JSON data. This patch basically does that. It also changes the uses of $c->validation, which are discouraged by the Mojolicious::Plugin::OpenAPI dev/maintainer. I do it to highlight what we must do in other places and the fact that there's no behavior change. To test: 1. Apply this patch 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/biblios.t \ xt/api.t => SUCCESS: Tests pass! 3. Sign off :-D -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #5 from Tomás Cohen Arazi --- Created attachment 148268 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148268=edit Bug 33227: Unit tests This patch adds a unit test that launches the swagger-cli validator against our spec. Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Attachment #148214|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Status|ASSIGNED|Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #4 from Martin Renvoize --- Confirmed working from this end so feel free to add my stamp if you're ready, Tomas. Otherwise, I'll wait for the bug to go NSO. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 --- Comment #3 from Martin Renvoize --- This is great! It'll be a vast improvement to have our spec validated regularly like this :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33227] OpenAPI validation is failing for paths/biblios.yaml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33227 Tomás Cohen Arazi changed: What|Removed |Added Summary|Swagger validation is |OpenAPI validation is |failing for |failing for |paths/biblios.yaml |paths/biblios.yaml -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/