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

store url of page the comment was posted on #22

Closed
gka opened this issue Oct 20, 2017 · 1 comment
Closed

store url of page the comment was posted on #22

gka opened this issue Oct 20, 2017 · 1 comment

Comments

@gka
Copy link
Member

gka commented Oct 20, 2017

this would be useful to have so we can link to the comment from the notification
we probably only need to store the path, as they would be all on the same domain (or maybe not?)

@jbutz
Copy link
Contributor

jbutz commented Sep 21, 2018

This may not even be necessary anymore since there is the URL in the config that gets the slug applied to it. The notifications just need to use it with the slug

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

No branches or pull requests

2 participants