Skip to content

ADR: API paths on run-config#89

Merged
delvh merged 1 commit intomainfrom
documentation/adr-api-paths
Jul 22, 2022
Merged

ADR: API paths on run-config#89
delvh merged 1 commit intomainfrom
documentation/adr-api-paths

Conversation

@madebyTimo
Copy link
Contributor

Approve with 1 for:

  • /api/service/vX/
    • e. g. /api/overworld/v1/config/minigame/

Approve with 2 for:

  • /service/api/vX/
    • e. g. /overworld/api/v1/config/minigame/

Approve with 3 for:

  • /api/vX/service/
    • e. g. /api/v1/overworld/config/minigame/

Copy link
Contributor

@delvh delvh left a comment

Choose a reason for hiding this comment

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

2

Copy link
Contributor

@maexled maexled left a comment

Choose a reason for hiding this comment

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

2

Copy link
Collaborator

@cdingm cdingm left a comment

Choose a reason for hiding this comment

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

2

Copy link
Member

@robojones robojones left a comment

Choose a reason for hiding this comment

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

🚒 2 pls

@delvh
Copy link
Contributor

delvh commented Jul 21, 2022

Time limit for commenting on/ reviewing this ADR: 80 minutes, until 23:59.

@delvh delvh changed the title add ADR API paths on run-config ADR: API paths on run-config Jul 21, 2022
@delvh
Copy link
Contributor

delvh commented Jul 22, 2022

Good, no new comments.
Merging now.

@delvh delvh merged commit 3abc2ca into main Jul 22, 2022
@delvh delvh deleted the documentation/adr-api-paths branch July 22, 2022 07:32
@madebyTimo
Copy link
Contributor Author

Ehm we forgot to write down the chosen solution

@delvh
Copy link
Contributor

delvh commented Jul 22, 2022

Oops. Follow up PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ADR This PR adds/ modifies an ADR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants