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

Move tracking script from within Cron Jobs #255

Open
wants to merge 4 commits into
base: 5.x
Choose a base branch
from

Conversation

nox1134
Copy link
Member

@nox1134 nox1134 commented Mar 15, 2024

Fixes #224 by @RCheesley
It appears that the tracking script instructions have been added to the cron jobs page. So, moving tracking script from within the cron jobs page to it's own page.
image
I'm not sure about what you mean by updating the redirect in the issue? (If it means updating in the readthedocs, then I think you need to do this through readthedocs dashboard)

Signed-off-by: Priyanshi Gaur <noxdot1134@gmail.com>
@RCheesley
Copy link
Sponsor Member

Thanks for the PR!

The note about the redirect was a 'note to self/team' that when this is done we'd need to redirect the old URL to the new location. You don't need to add those to the document, but instead please pop it in a comment here so we make sure to add the redirect on Read the Docs once the PR is merged!

If you can just remove that section we can then get this merged.

Signed-off-by: Priyanshi Gaur <noxdot1134@gmail.com>
Copy link
Sponsor Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

Good to go, thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move tracking script from within Cron Jobs
3 participants