Skip to content

Complete site's build settings. #61

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 2 commits into from
Dec 19, 2017
Merged

Complete site's build settings. #61

merged 2 commits into from
Dec 19, 2017

Conversation

calavera
Copy link
Contributor

The build settings is basically a repository into object underneath.
This changes the definition to reuse the RepoInfo struct. repo_branch
is used a read/write attribute to set the production branch for a site.

@mitchellh repo_type is a deprecated field and we don't use it for anything. This change should address the rest of attributes.
I renamed branch to repo_branch because it's only one property that you can use it for read and write.

Fixes #57.

Signed-off-by: David Calavera [email protected]

The build settings is basically a repository into object underneath.
This changes the definition to reuse the RepoInfo struct. `repo_branch`
is used a read/write attribute to set the production branch for a site.

Signed-off-by: David Calavera <[email protected]>
Signed-off-by: David Calavera <[email protected]>
@calavera calavera force-pushed the complete_build_settings branch from 6a13d32 to f6fb587 Compare December 19, 2017 04:52
@calavera calavera merged commit 1d2abfd into master Dec 19, 2017
@calavera calavera deleted the complete_build_settings branch December 19, 2017 04:56
@mitchellh
Copy link

Thanks! That's no problem and I noticed that there was some weird naming thing going on there but had no idea which way was the right way. I'll update this to work no problem. Thank you.

@mitchellh
Copy link

Hey @calavera I noticed the frontend sends branch, and you changed the API to use repo_branch as you said. I haven't verified it either way but is this okay? They both work?

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.

2 participants