europace / baufismart-antraege-api

As a loan provider get all the data of your applications for highly effective validations and approvals.
https://docs.api.europace.de/baufinanzierung/antraege/antraege-api/
Apache License 2.0
6 stars 5 forks source link

Update der Dokudatei(en) #180

Closed github-actions[bot] closed 1 year ago

saigyo commented 1 year ago

@unstko willst du hiermit schon das Release 2.64 bauen, oder wollen wir die Änderungen aus https://github.com/hypoport/antraege-edge-server/pull/633 mit ins 2.64 Release machen?

unstko commented 1 year ago

willst du hiermit schon das Release 2.64 bauen, oder wollen wir die Änderungen aus hypoport/antraege-edge-server#633 mit ins 2.64 Release machen?

@saigyo Ich würde hiermit schon das Release 2.64 erstellen. Dann haben wir je Thema ein eigenes Release mit Beschreibung der jeweiligen Änderungen.

saigyo commented 1 year ago

Mit der Neugenerierung der Swagger-Json im o.g. PR wird sowieso die Beschreibung der von Jan neu eingeführten Property nochmal mit dem aktualisierten, englischen Text überschrieben.

saigyo commented 1 year ago

willst du hiermit schon das Release 2.64 bauen, oder wollen wir die Änderungen aus hypoport/antraege-edge-server#633 mit ins 2.64 Release machen?

@saigyo Ich würde hiermit schon das Release 2.64 erstellen. Dann haben wir je Thema ein eigenes Release mit Beschreibung der jeweiligen Änderungen.

@unstko dann müsstest du die Swagger-JSON auf dem Stand nochmal neu generieren, weil die Beschreibung des neuen Attributs von Jan noch Deutsch ist.

saigyo commented 1 year ago

willst du hiermit schon das Release 2.64 bauen, oder wollen wir die Änderungen aus hypoport/antraege-edge-server#633 mit ins 2.64 Release machen?

Unserer Meinung nach musst du nicht zwingend für jede Änderung ein Release machen. Die beiden Änderungen könnten auch in einem Release gesammelt werden. Aber mach es so, wie du es für richtig hältst.

saigyo commented 1 year ago

@unstko siehe https://github.com/hypoport/antraege-edge-server/pull/627/files#diff-984d5c4d8e23b5c9a1f52db678394ef87a07518ed3e83e067e5f2b7e10425da8R2820

unstko commented 1 year ago

dann müsstest du die Swagger-JSON auf dem Stand nochmal neu generieren, weil die Beschreibung des neuen Attributs von Jan noch Deutsch ist. Unserer Meinung nach musst du nicht zwingend für jede Änderung ein Release machen. Die beiden Änderungen könnten auch in einem Release gesammelt werden. Aber mach es so, wie du es für richtig hältst.

@saigyo Danke, habe ich eben auch festgestellt. Dann lass uns doch nur ein Release für beide Anpassungen machen. Ich schließe diesen PR.