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

API improvements & storage/separation users #1816

Closed
wants to merge 3 commits into from

Conversation

alexalouit
Copy link

API & Gui return only current user data, if YOURLS is in public mode, « guest » user is use.
API: delete a shortlink with « delete » action (« url » request params, use keyword) owned by current user.
SQL: add user in « yourls_url » table, and index it.

Ability to filter by user on « stats » action (add « owner » request
params)
Ability to delete a shortlink (use keyword) with « delete » action («
url » request params).
API & Gui return only current user data, if YOURLS is in public mode,
« guest » user is use.
API:  delete a shortlink with « delete » action (« url » request
params, use keyword) owned by current user.
SQL: add user in « yourls_url » table, and index it.
@ozh
Copy link
Member

ozh commented Dec 4, 2014

This is a naive implementation of user support. For instance, if a user renames from "Joe" to something else, you'd have to change all the lines in the DB where user = 'joe' and update them.

Also, when we have users, we need roles, because obviously all users cannot be equal.

See #1678 and the Roadmap.

@ozh ozh closed this Dec 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants