This repository was archived by the owner on Dec 16, 2021. It is now read-only.
Repository navigation
add link and message to nodeschool upon successful installation #4 - #22
Merged
Merged
Conversation
azat-co
changed the title
azat-co
changed the title
Contributor
Contributor
|
This is a great start. I think that eventually we'll want to expand this but it's good to have this in there now so that we message the purpose of the installer a bit better as a vehicle for on-boarding. |
Contributor
|
There's a merge conflict tho :( |
… everything pretty
Contributor
Author
|
While there, I added node logo and made boxes have same width (prettier). |
Contributor
Author
|
@mikeal good or no good? |
Closed
Contributor
Author
|
@JoeDoyle23 thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
how about something like this for #4?