Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

Include token name and required scopes in PAT link #1010

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

EspadaV8
Copy link

@EspadaV8 EspadaV8 commented Jul 27, 2022

Description

Include a default name and required scopes in the URL to generate the personal access token.

Related Issue

Resolves #[issue_number]

How Has This Been Tested?

I haven't tested this via the code (not sure how to build it), but you can see it working by visiting

https://gitlab.com/-/profile/personal_access_tokens?name=GLab&scopes=api,write_repository

I just found out about GLab and added the feature to support passing in the name and scopes to the PAT page on GitLab. When setting it up I noticed they weren't present and could simplify the set up for other new users.

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)

@profclems
Copy link
Owner

profclems commented Aug 9, 2022

Hi @EspadaV8, thanks a lot for this PR. We announced that this project is officially adopted by GitLab #983 (comment). All new contributions should be mode to the GitLab repo: https://gitlab.com/gitlab-org/cli

@clemsbot
Copy link
Collaborator

clemsbot commented Sep 9, 2022

This PR has been automatically marked as stale because it has not had recent activity. This message is just a reminder for the author, reviewers and assignees.

@clemsbot clemsbot added the stale label Sep 9, 2022
ashumkin pushed a commit to ashumkin/glab that referenced this pull request Sep 6, 2023
docs: convert documentation to markdown

Closes profclems#1010

See merge request https://gitlab.com/gitlab-org/cli/-/merge_requests/1041

Merged-by: Gary Holtz <[email protected]>
Approved-by: Ahmed Hemdan <[email protected]>
Approved-by: Amy Qualls <[email protected]>
Co-authored-by: Kai Armstrong <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants