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

Feature: Add MVP for Organisation Page #1

Merged
merged 5 commits into from Jul 18, 2022

Conversation

ChargrilledChook
Copy link
Member

@ChargrilledChook ChargrilledChook commented Jul 8, 2022

Discussed in TheOdinProject/top-meta#191

This is a basic MVP version for an org landing page while we flesh it out with some content. It won't show on the org page until moved to the profile directory.

Feel free directly suggest any changes in the markdown with review suggestions.

Resolves TheOdinProject/top-meta#191

Copy link
Contributor

@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

Just a couple of comments for now, nothing that really needs immediate action. To continue the conversation here (some repeat comments, but just to keep it all here), I personally think this landing page should include:

  • A section for our curriculum repo that explains how it is divided into three distinct paths (this can be easily update if any additional paths are ever added), with each path having courses that cover more specific areas. I think keeping the verbiage about it being a project based curriculum is worth it as well. Finally, a line about why you would/what kind of things you woudl contribute to the repo.
  • A section about the main TOP site (I think what you had as the draft works), with a line about why you would/what kind of things you woudl contribute to the repo.
  • I think the Community section can work, it should just be more brief
  • The section about how to contribute

README.md Outdated


![odin landing image](https://www.theodinproject.com/assets/home-isometric-eebc110218cbdb1460dd2dfc662a4e96ae94747667dc81da422804ef55cfd2d8.svg)
A special repo for hosting community health files and our ogranisation profiles.
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this is just copy+pasted, but is this top-level README required, or could we delete it and just have the profile directory?

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed that the original was low value but I've gone the other way and made it more detailed to document usage since it's not immediately obvious

@@ -0,0 +1,6 @@
# Welcome To The Odin Project!

[The Odin Project (TOP)](https://www.theodinproject.com/) is an open-source curriculum for learning full-stack web development. Our mission is to provide a comprehensive curriculum to learn web development for free. We help users learn the skills and build the impressive portfolio of projects they need to get hired as a web developer.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think until we can align on a mission statement, we could just have for now:

Suggested change
[The Odin Project (TOP)](https://www.theodinproject.com/) is an open-source curriculum for learning full-stack web development. Our mission is to provide a comprehensive curriculum to learn web development for free. We help users learn the skills and build the impressive portfolio of projects they need to get hired as a web developer.
[The Odin Project (TOP)](https://www.theodinproject.com/) is an open-source curriculum for learning full-stack web development. We help users learn the skills and build the impressive portfolio of projects they need to get hired as a web developer.

Though from there, do we want to limit this verbiage to only "web development", or go for a more general "programming" or rather "software engineer"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done, this was lifted from the README of the app repo, but happy to take out mission.

Though from there, do we want to limit this verbiage to only "web development", or go for a more general "programming" or rather "software engineer"?

I think web developer is fairly descriptive / representative of the current state of the curriculum and the path most learners would take, but I could see some value in making it more general.

@ChargrilledChook ChargrilledChook merged commit ad793bb into main Jul 18, 2022
@ChargrilledChook ChargrilledChook deleted the feature/add-organisation-landing-page branch July 18, 2022 13:42
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.

Feature: Add landing page to Odin Project Org on GitHub
2 participants