Svoboda | Graniru | BBC Russia | Golosameriki | Facebook
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

Fix broken courseware page link. #227

Merged
merged 56 commits into from
Aug 5, 2022
Merged

Fix broken courseware page link. #227

merged 56 commits into from
Aug 5, 2022

Conversation

juliawu
Copy link
Contributor

@juliawu juliawu commented Aug 4, 2022

Fixes the link to the courseware page in our contributing page that currently leads to a 404 error.

@juliawu juliawu linked an issue Aug 4, 2022 that may be closed by this pull request
Copy link
Contributor

@chejennifer chejennifer left a comment

Choose a reason for hiding this comment

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

thanks for the fix!

@shifucun
Copy link
Contributor

shifucun commented Aug 4, 2022

Just a minor comment. For a PR, it's good to start from a clean new local branch so the commit history only contains the related change for this PR.

@juliawu juliawu merged commit 9a6b65a into datacommonsorg:master Aug 5, 2022
@juliawu
Copy link
Contributor Author

juliawu commented Aug 5, 2022

Just a minor comment. For a PR, it's good to start from a clean new local branch so the commit history only contains the related change for this PR.

Thanks for the tip, will do for future PRs!

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.

Broken Courseware Link on Web site
4 participants