-
Notifications
You must be signed in to change notification settings - Fork 48
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
Update welcome package info #64
base: main
Are you sure you want to change the base?
Update welcome package info #64
Conversation
Update welcome package info
✅ Deploy Preview for strong-fairy-c1bde1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just need to make some updates to the team image to fix some issues with student names in the SplashKit team.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update 2 of the SplashKit team member names that are incorrect.
"Hayden Hughes" should be "Hayley Hughes", and "James Stuart Balsillie" should be "Jess Balsillie".
I also suggest swapping my name and Tien's name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Eisen666 I suggest removing this file from this PR and then update the onboarding document to use the corrected image from this PR: https://github.com/thoth-tech/ThothTech-Documentation-Website/pull/67/files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, got it. I will finish before tomorrow afternoon.
update the chart
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice.
Update welcome package info
Update welcome package info
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Already tested on the locally machine.
Testing Checklist