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

Update integrations/scm_repo_link structs and urls #125

Closed
brandon-welsch opened this issue Jul 31, 2019 · 0 comments · Fixed by #126
Closed

Update integrations/scm_repo_link structs and urls #125

brandon-welsch opened this issue Jul 31, 2019 · 0 comments · Fixed by #126
Assignees
@brandon-welsch

Comments

@brandon-welsch
Copy link
Member

brandon-welsch commented Jul 31, 2019 •
edited
Loading

Update theses modifications to keep a functional coherence:

  • Auth Service has a new updated route for integrations: /integrations -> /scm_integrations
  • API has renamed a field in scm_repo_link: auth_integration_id -> auth_integration_uuid

Add a missed method for the SCMType constant:

func (t SCMType) Str() string {
	return string(t)
}

And rename all Scm... to SCM...
@brandon-welsch brandon-welsch self-assigned this Jul 31, 2019
@brandon-welsch brandon-welsch changed the title Update integrations/scm_repo_link struct and calls Update integrations/scm_repo_link struct and urls Jul 31, 2019
@brandon-welsch brandon-welsch changed the title Update integrations/scm_repo_link struct and urls Update integrations/scm_repo_link structs and urls Jul 31, 2019
@brandon-welsch brandon-welsch mentioned this issue Jul 31, 2019
Merged
@EtienneM EtienneM mentioned this issue Sep 24, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brandon-welsch brandon-welsch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated integrations/scm_repo_link structs and urls Scalingo/go-scalingo
1 participant
@brandon-welsch