-
Notifications
You must be signed in to change notification settings - Fork 6
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
Additional student guidance #18
Additional student guidance #18
Conversation
Looks good! :) Happy to merge this or if you want to make further changes here following today's meeting then go for it! |
952ffa3
to
5ccc208
Compare
@l-gorman as discussed, I've suggested some (minor) changes. Happy for you to continue with the other things you think are needed:
...with the links to SharePoint, Tracker, Slack, etc. I wonder if it's worth linking to a (newly-created) single page that links to everything external?
(See what you think) I've added @RichardP1234 to review in my absence :) |
Based on our group discussion today, we're going to merge this PR as-is now, and open up new issues for any remaining work. :) Thanks @l-gorman and @RichardP1234 for your help! 🎉 |
Thanks @jatonline! Sorry I didn't get round to the final changes 🙌 |
Draft PR to implement changes which need to
be made before the new members of ask JGI team joint
Further updates to add #19