Skip to content
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

Updated Content for ml5.js next gen #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

shiffman
Copy link
Member

First, apologies, but my VSCode settings adjusted many small style elements in the codebase, we should create .vscode directory with workspace settings for this project so we can keep it uniform. This is a first pass at editing some language that needs refreshing from the previous site.

Some misc notes and questions:

  • I removed the Code of Conduct from the site b/c it is the older language before the creation of the newer Code of Conduct which lives in its own repo. So I'm linking to that, but perhaps some quotes or overall language would still be good to keep.
  • For elements that are "coming soon", should we remove these or do we have some initial content we can add? Personally, I might prefer to temporarily remove and then add them in when ready (over having "coming soon" on the site.)
  • @sproutleaf and @acosmica, it might be worth checking out this google form and seeing if it still applies or if we want to change it?
  • We need to add updated credits about the website, maybe it should also link to the all contributors part of the GitHub repo?
  • We need to add updated information about the collaboration between NYU Shanghai and NYU NY, as well as DAIL Center credit #8 as noted by @gohai.

We're so close to launching the site, yay!

Perhaps this list should get broken out into issues for future development? I'll be reviewing the docs site next!

Copy link

netlify bot commented Apr 23, 2024

Deploy Preview for ml5-next-gen ready!

Name Link
🔨 Latest commit 24e7103
🔍 Latest deploy log https://app.netlify.com/sites/ml5-next-gen/deploys/662838263439e90008f49ac4
😎 Deploy Preview https://deploy-preview-9--ml5-next-gen.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@shiffman
Copy link
Member Author

Whooops, I found this old pull request when looking at some parts of the site. I think everything here was addressed in separate updates and changes so it probably can be closed? Feel free to take any other edits if you like!

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.

1 participant