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

updating the tdm-calculater.md and pdf #2801

Closed
wants to merge 1 commit into from
Closed

updating the tdm-calculater.md and pdf #2801

wants to merge 1 commit into from

Conversation

blulady
Copy link
Member

@blulady blulady commented Feb 19, 2022

Fixes #replace_this_text_with_the_issue_number

What changes did you make and why did you make them ?

-Downloaded a copy of LA TDM Calculator's overview pdf and put it in [/assets/pdfs]
-Changed the 'Overview' link for the [markdown file]
-Ensured that the links in both the project page and the project's [dedicated page]

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image (54)

Visuals after changes are applied

image(55)

@github-actions
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b blulady-update-overview-link-TDM-Calc-2211 gh-pages
git pull https://github.com/blulady/website.git update-overview-link-TDM-Calc-2211

Copy link
Member

@SAUMILDHANKAR SAUMILDHANKAR left a comment

Choose a reason for hiding this comment

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

@blulady I understand the changes that you want to make are related to issue #2242, however, since your PR #2772 is supposed to fix the issue, so the changes should be made there. Please close this PR and continue working on PR #2772. Thank you.

@blulady blulady closed this Feb 20, 2022
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.

2 participants