Skip to content
This repository has been archived by the owner on Sep 24, 2022. It is now read-only.

Add links to about pages of current templates #243

Merged

Conversation

Linda-Ikechukwu
Copy link
Contributor

I added links to the individual about pages of each template type listed in the Readme for easier navigation.
This way, users who land on the readme can easily locate the individual templates without having to dig through files in the repo.

I think the next thing to do would be to add links to each template file in their respective about pages too.

I added links to the individual about pages of each template type listed in the Readme for easier navigation. 
This way, users who land on the readme can easily locate the individual templates without having to dig through files in the repo. 

I think the next thing to do would be to add links to each template file in their respective about pages too.
@barbaricyawps barbaricyawps self-requested a review March 22, 2022 20:24
@barbaricyawps
Copy link
Contributor

Hi, @Linda-Ikechukwu . Congratulations on your first pull request to our community!

I'm glad you're doing this great work because it's needed. I'd also like to invite you to join some of our community spaces to talk to some of the other project contributors to get some greater context about what we're doing with the templates. For example, we're planning a summit in the future where we'll re-organize our templates fairly substantially and it would be great to possibly invite you to that discussion. It might also be helpful to have you join our Content Strategy working group.

This page explains how to join our community and begin collaborating with us: Community. In particular, I recommend joining our Slack workspace.

Let me know if you have any additional questions and I'll do a deeper review this weekend!

@Linda-Ikechukwu
Copy link
Contributor Author

Hi @barbaricyawps That'd be so cool. I'll go through the documents and ping you on slack if I can find you :)

Copy link
Member

@camerons camerons left a comment

Choose a reason for hiding this comment

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

Thanks Linda for suggesting links. It is a good idea.
It will likely be slightly better to use relative links. That way if we change the repository name, which we are planning to do, the links will still work.
I've suggested a couple of changes. Could you please update the rest. Please also check my suggestions work. (I haven't.)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
barbaricyawps and others added 2 commits April 17, 2022 14:24
Co-authored-by: Cameron Shorter <[email protected]>
Co-authored-by: Cameron Shorter <[email protected]>
Copy link
Contributor

@barbaricyawps barbaricyawps left a comment

Choose a reason for hiding this comment

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

I've committed @camerons 's changes. This contribution looks great! I'm merging this in.

Thanks, @Linda-Ikechukwu !

@barbaricyawps barbaricyawps merged commit bb8bcec into thegooddocsproject:dev Apr 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants