Skip to content

Add GitLab URL field to user profile #1121

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

Merged
merged 5 commits into from
Nov 15, 2018

Conversation

sidemt
Copy link
Contributor

@sidemt sidemt commented Nov 15, 2018

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Added GitLab URL field to user profile page and corresponding database/model/controllers.
This should close #1080 .

Regarding the validation of the URL, as far as I checked the GitLab's sign up form it does not have a limitation on the length of the username and it can contain . or - in it.

Related Tickets & Documents

#1080

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

gitlab_url
gitlab_url2

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Nov 15, 2018
@CLAassistant
Copy link

CLAassistant commented Nov 15, 2018

CLA assistant check
All committers have signed the CLA.

@pr-triage pr-triage bot removed the PR: unreviewed bot applied label for PR's with no review label Nov 15, 2018
@benhalpern benhalpern merged commit 62d8a33 into forem:master Nov 15, 2018
@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Nov 15, 2018
@benhalpern
Copy link
Contributor

Hey @sidemt, this is live. Feel free to make a changelog post like these ones if you want:

https://dev.to/t/changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add GitLab field to Profile
3 participants