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: Re-add hash and outbound-links only options #655

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Fix: Re-add hash and outbound-links only options #655

merged 1 commit into from
Jan 26, 2021

Conversation

jonnitto
Copy link

Changes

Please describe the changes made in the pull request here.

Below you'll find a checklist. For each item on the list, check one option and delete the other.

Tests

  • This PR does not require tests

Changelog

  • This PR does not make a user-facing change

Documentation

  • This change does not need a documentation update

@jonnitto
Copy link
Author

jonnitto commented Jan 26, 2021

hash and outbound-links only option was removed with this merge:

fb7a3fe#diff-4937d77a31579beff9907cd9e7f57d322bb765e6f55c5934641d50ddbdfcac19

@metmarkosaric metmarkosaric merged commit 66735f4 into plausible:master Jan 26, 2021
@Vigasaurus
Copy link
Contributor

Huh, I dunno how I did that/didn't notice, heh

@metmarkosaric
Copy link
Contributor

thanks @jonnitto! appreciate your help!

@jonnitto
Copy link
Author

jonnitto commented Jan 27, 2021

It was a pleasure!

I was happy for my integration, because I got there an error like this:

CleanShot 2021-01-27 at 11 21 04@2x

@jonnitto
Copy link
Author

I think it would be good to add also all the variants of scripts to a monitor system. Wdyt?

@ukutaht
Copy link
Contributor

ukutaht commented Jan 27, 2021

@jonnitto Makes sense I will add it to our uptime monitoring

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.

4 participants