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

ZO-4598: add 'summy' API endpoints #124

Merged
merged 3 commits into from
Mar 26, 2024
Merged

ZO-4598: add 'summy' API endpoints #124

merged 3 commits into from
Mar 26, 2024

Conversation

kabbeck
Copy link
Contributor

@kabbeck kabbeck commented Mar 15, 2024

API Config für 'summy' "summaries" Endpunkt.

@kabbeck kabbeck marked this pull request as ready for review March 19, 2024 11:03
@kabbeck kabbeck requested a review from witsch March 19, 2024 11:03
Copy link
Contributor

@witsch witsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich hab mal versucht, openapicmd zu benutzen, um die aktuelle Spec aus 'summy' mit dieser hier zu vergleichen (mit openapi read --dereference …), und mich dann gefragt, wie wir die am einfachsten "übereinander kriegen". Oder wird das nichts, weil wir verschiedene OpenAPI Versionen haben? Da sollten wir vielleicht demnächst™ nochmal in Ruhe gucken, wie wir sowas am besten angehen können…

Für den Moment sieht das soweit aber prima aus. Eventuell sollten wir noch kurz den Umbau wegen des Feedbacks (auf verschiedene Versionen) abwarten, weil sich dadurch ja die Properties nochmal ändern. Aber das können wir auch noch nachschieben…

@witsch
Copy link
Contributor

witsch commented Mar 22, 2024

Für den Moment sieht das soweit aber prima aus. Eventuell sollten wir noch kurz den Umbau wegen des Feedbacks (auf verschiedene Versionen) abwarten, weil sich dadurch ja die Properties nochmal ändern. Aber das können wir auch noch nachschieben…

Also konkret die Änderungen aus https://github.com/ZeitOnline/summy/pull/26

@witsch witsch changed the title ZO-4598 ZO-4598: add 'summy' API endpoints Mar 26, 2024
@witsch witsch merged commit 07f8a2b into main Mar 26, 2024
1 check passed
@witsch witsch deleted the ZO-4598 branch March 26, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants