Skip to content

Conversation

colinl
Copy link
Contributor

@colinl colinl commented Nov 10, 2020

Moved boxes for Pi and RPM up to the top and added one for Windows.
Removed Windows link from box under Installing with npm.

@@ -45,7 +52,6 @@ sudo npm install -g --unsafe-perm node-red
<img src="/images/logos/windows.svg" height="30">
Copy link
Member

Choose a reason for hiding this comment

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

Not sure we need this be a callout box any more as you now have that above. I think normal text will do.

@dceejay
Copy link
Member

dceejay commented Nov 10, 2020

Not sure we now need the second part to be a callout - I think plain text will do

@colinl
Copy link
Contributor Author

colinl commented Nov 10, 2020

OK, will do. There is another box the same down at the bottom of the page, I will do the same there.

@colinl
Copy link
Contributor Author

colinl commented Dec 7, 2020

Is there a reason this has not been merged, or is it just pressure of other work?

@knolleary
Copy link
Member

@colinl sorry - thought this had already been merged. Thanks for the nudge.

@knolleary knolleary merged commit 586fc04 into node-red:master Dec 7, 2020
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.

3 participants