datacarpentry / python-ecology-lesson

Data Analysis and Visualization in Python for Ecologists
https://datacarpentry.org/python-ecology-lesson
Other
162 stars 308 forks source link

Link redirect on line 23? #397

Closed doujouDC closed 5 years ago

doujouDC commented 5 years ago

Hi, on line 23, I get the follow error when I click on the "follow the directions" link: Oops! Nothing here...

Please delete this line and the text below before submitting your contribution.


Thanks for contributing! If this contribution is for instructor training, please send an email to checkout@carpentries.org with a link to this contribution so we can record your progress. You’ve completed your contribution step for instructor checkout just by submitting this contribution.

Please keep in mind that lesson maintainers are volunteers and it may be some time before they can respond to your contribution. Although not all contributions can be incorporated into the lesson materials, we appreciate your time and effort to improve the curriculum. If you have any questions about the lesson maintenance process or would like to volunteer your time as a contribution reviewer, please contact The Carpentries Team at team@carpentries.org.


maxim-belkin commented 5 years ago

Thank you for the report, Doug. It should be bringing you to https://datacarpentry.org/python-ecology-lesson/setup.html (but it does not). I think I know the reason why (the problem is spefici to data-carpentry lessons).

Before we fix it globally, could you please change [lesson-setup] to ({{page.root}}{% link setup.md %}) in this PR? (Also, please wrap the lines to not exceed 100-character limit.) :)

maxim-belkin commented 5 years ago

@fmichonneau, to make links that use {{relative_root_path}} work, we need to add {% include base_path.html %} to every file that uses them. Is there a way to make every episode auto-include this base_path.html?

maxim-belkin commented 5 years ago

Actually, looks like all we have to do is add {% include base_path.html %} to links.md file...

maxim-belkin commented 5 years ago

I created a PR upstream (carpentries/styles#405) that should resolve this issue.

doujouDC commented 5 years ago

Hey, this was my first pull-request (yikes) so I want to make sure I do not need to do anything else. #newbie

maxim-belkin commented 5 years ago

I apologize for rush-fixing this issue myself. I usually don't do that but there are two reasons I did it this time:

  1. This bug is kind of a "show-stopper" in my view so I wanted to fix it ASAP. The origin of the problem is in the styles repository (that we merge into this one from time to time).
  2. This PR is basically a report of the problem -- we use Issues for that. I would be happy to help you with your next PR, if you're OK with that. Otherwise, just tag April (@wrightaprilm) and she'll chime in when she can.
doujouDC commented 5 years ago

Hey,

Not worries...I am not sure I would even know how to fix this on my own.

Cheers

Doug

On Wed, May 29, 2019 at 5:56 PM Maxim Belkin notifications@github.com wrote:

I apologize for rush-fixing this issue myself. I usually don't do that but there are two reasons I did it this time:

  1. This bug is kind of a "show-stopper" in my view so I wanted to fix it ASAP. The origin of the problem is in the styles https://github.com/carpentries/styles repository (that we merge into this one from time to time).
  2. This PR is basically a report of the problem -- we use Issues https://github.com/datacarpentry/python-ecology-lesson/issues for that. I would be happy to help you with your next PR, if you're OK with that. Otherwise, just tag April (@wrightaprilm https://github.com/wrightaprilm) and she'll chime in when she can.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/datacarpentry/python-ecology-lesson/pull/397?email_source=notifications&email_token=ABRKTFCYFFZKCOVWMK5YCRTPX332NA5CNFSM4HQPD3T2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWQYCVY#issuecomment-497123671, or mute the thread https://github.com/notifications/unsubscribe-auth/ABRKTFCTRYHKT3EABYCSGHTPX332NANCNFSM4HQPD3TQ .