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

Adjust /me redirection to /settings/tokens #4311

Merged
merged 1 commit into from
Dec 22, 2021
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Dec 22, 2021

As discussed on Discord, the cargo login command is showing links to https://crates.io/me to access the API tokens. Since that is probably more important than any other bookmarks and has backwards compatibility concerns this PR is changing the /me redirection to /settings/tokens (instead of /settings aka. /settings/profile).

@Turbo87 Turbo87 added A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works labels Dec 22, 2021
@Turbo87 Turbo87 requested review from locks and pichfl December 22, 2021 09:23
@locks
Copy link
Contributor

locks commented Dec 22, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 22, 2021

📌 Commit 13b08fa has been approved by locks

@bors
Copy link
Contributor

bors commented Dec 22, 2021

⌛ Testing commit 13b08fa with merge e4e3802...

@bors
Copy link
Contributor

bors commented Dec 22, 2021

☀️ Test successful - checks-actions
Approved by: locks
Pushing e4e3802 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants