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

Website Overhaul #85

Merged
merged 11 commits into from
Aug 13, 2024
Merged

Website Overhaul #85

merged 11 commits into from
Aug 13, 2024

Conversation

ShaunR1991
Copy link
Collaborator

Description

  • Updated splash page:

  • Fixed broken links

  • Adjusted the About Us language

  • Updated quick links to more popular pages:

    1. Site Introduction
    2. Onboarding Hub (content WIP)
    3. Documentation
    4. News
  • Content pages restructured to group like pages for ease of navigation.

  • Updates "Processes" tab

  • Fixed all broken links

  • Overall improved language to be more concise, gramattically correct, free of spelling errors and correct punctuation.

  • Renamed to "Resources" to be more appropriate for content

  • Updated language in the introduction

  • Validated and expanded on the technology stack documentation

  • Removed "Cyber Security Guidelines" parent folder to have the "Remote Working Guide" unnested.

  • Created Onboarding Hub

  • Moved the Thoth Tech Welcome Package to here from "Policies"

  • Created placeholders for current teams

  • Updated Quality Assurance Overview document language

  • Validated and updated the Git Contributions Guide.

  • Added explanations to commands

  • Expanded some explanations

  • Improved overall language

  • Updated Testing and Development content

  • Updated Documentation Tab

  • Documenting for Thoth Tech document format adjusted and languate improved.

  • Writing Style Guide formatting improved; examples provided for all items.

  • Templates moved to Documentation tab for coherence

  • FAQs moved to Resources; added a few additional FAQs.

  • Consolidated Teams and Leadership page into trimesters.

  • Included current trimester information into Teams and Leadership page.

  • Consolidated Company Submissions into appropriate trimesters for ease of navigation.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (update or new)

How Has This Been Tested?

Tested in Astro dev environment with no new errors.
Tested that all links work.

Testing Checklist

  • Tested in latest Chrome
  • Tested in latest Safari
  • Tested in latest Firefox

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have requested a review from SplashKit Leadership Team on the Pull Request

Copy link

netlify bot commented Jul 26, 2024

Deploy Preview for strong-fairy-c1bde1 ready!

Name Link
🔨 Latest commit 1917ccb
🔍 Latest deploy log https://app.netlify.com/sites/strong-fairy-c1bde1/deploys/66aea4ff52b5850008c5cc74
😎 Deploy Preview https://deploy-preview-85--strong-fairy-c1bde1.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.

Copy link
Contributor

@breezy-codes breezy-codes left a comment

Choose a reason for hiding this comment

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

This is great, the site looks much better with the new organisation and the additions of the new team lists for this semester.

Copy link
Contributor

@Liquidscroll Liquidscroll left a comment

Choose a reason for hiding this comment

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

Great work! I really like how you've organised it, it's definitely given the website a professional refresh.

There's a few small changes to be made.

The other issue with this, however, is the same as in your other PR (#87). They both include the same changes and will conflict when we merge these in. This will need to be fixed.

Yes, there is. Find some helpful links below:

- [Instructional video](https://deakin365.sharepoint.com/:v:/r/sites/ThothTech2/Shared%20Documents/General/Instructional%20Video%20-%20How%20To%20Upload%20Documents%20to%20Starlight.mp4?csf=1&web=1&e=s26lMo).
- [Help hub session recording](https://deakin365-my.sharepoint.com/:v:/g/personal/s221460489_deakin_edu_au/EXib27xJI2hCkHpzBXOxxYQBOfDGhhQJ1A23ciisMROeRA?xsdata=MDV8MDF8fDgxMTZlNDYzMzAyZTQxYTZhNGE2MDhkYmYyMWMwMGY1fGQwMjM3OGVjMTY4ODQ2ZDU4NTQwMWMyOGI1ZjQ3MGY2fDB8MHw2MzgzNjk5NzQyOTIxNjE3MDJ8VW5rbm93bnxWR1ZoYlhOVFpXTjFjbWwwZVZObGNuWnBZMlY4ZXlKV0lqb2lNQzR3TGpBd01EQWlMQ0pRSWpvaVYybHVNeklpTENKQlRpSTZJazkwYUdWeUlpd2lWMVFpT2pFeGZRPT18MXxMM1JsWVcxekx6RTVPbEZtZUY5VFZFaFZPVEJQYzFaWlFraFdXVXRvYzFKUk5XZHRSV1V3Y3psUk5tdFBjRUptTm1Kc2FUZ3hRSFJvY21WaFpDNTBZV04yTWk5amFHRnVibVZzY3k4eE9UcFJabmhmVTFSSVZUa3dUM05XV1VKSVZsbExhSE5TVVRWbmJVVmxNSE01VVRaclQzQkNaalppYkdrNE1VQjBhSEpsWVdRdWRHRmpkakl2YldWemMyRm5aWE12TVRjd01UUXdNRFl5T0RNd01nPT18NTg0ZjIwNWZmZjA3NGZlZTQyNjQwOGRiZjIxYzAwZjN8ZTU2N2Y4NzA4YjFlNDI1YWIzYjRhM2I1NmExY2QwMjE%3D&sdata=TldtNGVnTDJWWlVnb2lBMGlZZXlLNEI1M2p5emU3ZndBYmpSTkV6MnN0QT0%3D).
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't have access to this video, is this just a permission issue for me or can anyone else view this video?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I can't access that video either

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've fixed this.

There was a problem where the link from Teams (i.e. Sharepoint) would return a 404 or permission error.

I've migrated this video over to Panopto which should now work.

- Jake Mansfield - Student Lead | OnTrack CourseFlow Development
- Jack Carroll - Student Lead | OnTrack JPLag Implementation
- Atharv Bhandare - Junior Leader | OnTrack JPLag Implementation
- Isabelle Holt - Senior Leader | OnTrack FrontEnd Migration
Copy link
Contributor

Choose a reason for hiding this comment

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

AFAIK Isabelle Holt isn't a leader anymore

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated.

| Simon Zhu | Team Member – Games Development | Senior |
| Brianna Laird | Junior Lead – SplashKit Tutorials & Expansion | Junior |
| Shaun Ratcliff | Junior Lead – SplashKit Website & Tutorials | Junior |
Jordan Brown | Team Member – SplashKit Website | Junior |
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing |

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed :)

@Liquidscroll Liquidscroll mentioned this pull request Aug 3, 2024
13 tasks
Added missing pipe "|" to table formatting.
Updated student leadership team.
Updated leadership team.
…als/building-vector-2d-shooter-proposal.md

Proposal was reviewed and included in PR87 - does not fit the scope of this PR.
Updated link to github workshop to be enduring (as opposed to ephemeral sharepoint/teams link)
Copy link
Collaborator Author

@ShaunR1991 ShaunR1991 left a comment

Choose a reason for hiding this comment

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

Updated leadership team
Fixed formatting on team member tables
Updated the FAQ video link to the workshop to be more enduring by making use of Deakin Panopto.
Fixed conflict with other PR (#87)

Copy link
Contributor

@Liquidscroll Liquidscroll left a comment

Choose a reason for hiding this comment

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

Great work, particularly fixing the merge conflicts!

Copy link
Collaborator

@omckeon omckeon left a comment

Choose a reason for hiding this comment

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

Love the new folders for the left sidebar! Particular the Resources folder :)

@omckeon omckeon merged commit 1917ccb into thoth-tech:main Aug 13, 2024
4 of 5 checks passed
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