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.

Sign up for GitHub

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

Jump to bottom

Convert /accounts to /profiles/register #168

Closed
tmichalski opened this issue May 12, 2018 · 1 comment
Closed

Convert /accounts to /profiles/register #168

tmichalski opened this issue May 12, 2018 · 1 comment
Assignees
@raymer
Labels
enhancement

Comments

@tmichalski
Copy link

tmichalski commented May 12, 2018 •
edited
Loading

To keep the API consistent, rework the endpoint /accounts POST (or any other method) to live under /profiles. The only two that appear to be needed are

/profiles/register POST - create a user record (PUBLICLY ACCESSIBLE, BE CAUTIOUS)
/profiles/me GET - fetch the current user's (login required, get currently logged in user)
/profiles/me PUT - update the current users profile (login required, get currently logged in user)
/profiles/me DELETE - logically deletes the current user and logs out the user
@tmichalski
Copy link
Author

Fixed in #170

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

@raymer raymer

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tmichalski @raymer