Skip to content

San 6010 private registry#2195

Merged
runnabro merged 14 commits intomasterfrom
SAN-6010-private-registry
Apr 19, 2017
Merged

San 6010 private registry#2195
runnabro merged 14 commits intomasterfrom
SAN-6010-private-registry

Conversation

@runnabro
Copy link
Copy Markdown
Member

@runnabro runnabro commented Apr 18, 2017

@runnabro runnabro requested a review from taylordolan April 18, 2017 17:57
spellcheck = "false"
type = "url"
)
small.grid-block.padding-xxs.small Add the URL for the registry you use. This could be from Docker Hub, or a private registry.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spacing would be more even if you removed the padding-bottom from this element.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I don't think you need the comma after "Docker Hub".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the default for all .small text under form inputs, I'm reluctant to change that right now.

spellcheck = "false"
type = "password"
)
footer.margin-top-sm.padding-sm.grid-block.justify-right.footer
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you decide against the "Cancel" or "Back" button here?

@@ -0,0 +1,78 @@
section.label-description
.label-col Private Registry
.small.text-gray If you push your images to a private registry, you can safely store your credientials to use those images on Runnable. 
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

credientials -> credentials

@runnabro runnabro merged commit ebf3259 into master Apr 19, 2017
@runnabro runnabro deleted the SAN-6010-private-registry branch April 19, 2017 22:01
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