Skip to content

remove resources page#4803

Merged
Trott merged 3 commits intonodejs:mainfrom
Trott:resources
Sep 2, 2022
Merged

remove resources page#4803
Trott merged 3 commits intonodejs:mainfrom
Trott:resources

Conversation

@Trott
Copy link
Copy Markdown
Member

@Trott Trott commented Aug 29, 2022

The site is not the place to make Illustrator files of the logo
available.

The site is not the place to make Illustrator files of the logo
available.
@nschonni
Copy link
Copy Markdown
Member

Are these available somewhere else, like the foundation site?

@Trott
Copy link
Copy Markdown
Member Author

Trott commented Aug 30, 2022

Are these available somewhere else, like the foundation site?

Open to reasons why I'm wrong about this, but I think as long as the marketing folks etc. at the Foundation have the files, I'm not sure other people should be messing with the logos. @rginn

@rginn
Copy link
Copy Markdown
Contributor

rginn commented Aug 30, 2022

All OpenJS project logos are kept up-to-date in our foundation artwork repo https://github.com/openjs-foundation/artwork, including Node.js.

@Trott Trott requested a review from joesepi August 30, 2022 20:26
Copy link
Copy Markdown
Member

@joesepi joesepi left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Do we need to refer the link in detail somewhere in our project?
https://github.com/openjs-foundation/artwork/tree/main/projects/nodejs#nodejs-artwork

@Trott
Copy link
Copy Markdown
Member Author

Trott commented Aug 31, 2022

Do we need to refer the link in detail somewhere in our project? https://github.com/openjs-foundation/artwork/tree/main/projects/nodejs#nodejs-artwork

I'm not opposed, but I don't know that we need it, to be honest. I know Python has it on their site, but I'm just not really sure who it's for, to be honest.

@Trott
Copy link
Copy Markdown
Member Author

Trott commented Sep 2, 2022

Landing. We can always revert if it turns out this is needed. I doubt it is, though.

@Trott Trott merged commit bc07228 into nodejs:main Sep 2, 2022
@Trott Trott deleted the resources branch September 2, 2022 00:18
@mcollina mcollina mentioned this pull request Jul 25, 2023
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.

4 participants