Skip to content

add ID prefixing to endpoints. - #178

Open
auroter wants to merge 8 commits into
developfrom
task/sean/str-556
Open

add ID prefixing to endpoints.#178
auroter wants to merge 8 commits into
developfrom
task/sean/str-556

Conversation

@auroter

@auroter auroter commented Apr 24, 2023 •
edited by ocasta181
Loading

Copy link
Copy Markdown
Contributor

To test:

  1. Hit all endpoints
  2. Ensure that any Ids in the JSON body show a prefix
  3. Ensure that any endpoints which accept Ids accept the prefixed Id without error

ocasta181
ocasta181 reviewed Apr 24, 2023
View reviewed changes
Comment thread api/handler/user.go Outdated
Comment thread go.mod Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ocasta181 ocasta181 ocasta181 left review comments
@saito-sv saito-sv Awaiting requested review from saito-sv
@alcalawil alcalawil Awaiting requested review from alcalawil
@frostbournesb frostbournesb Awaiting requested review from frostbournesb

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@auroter @ocasta181