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

Error when submitting a comment on a recently deleted post #6610

Closed
adamkdean opened this issue Mar 12, 2020 · 11 comments · Fixed by #13721
Closed

Error when submitting a comment on a recently deleted post #6610

adamkdean opened this issue Mar 12, 2020 · 11 comments · Fixed by #13721
Assignees
Labels
area: publishing experience issues related to an authors experience publishing. Tags, series, etc. bug smash Approved bugs for the DEV community bug smash bug always open for contribution external contributors welcome contribution is welcome!

Comments

@adamkdean
Copy link

adamkdean commented Mar 12, 2020

Describe the bug

Followed a link from an email, attempted to respond to a comment, received error There was an error in your markdown: undefined method id' for nil:NilClass`, comment not posted.

To Reproduce

Follow link from an email, attempt to respond to a comment.

Expected behavior

Comment is posted.

Screenshots

image

Desktop (please complete the following information):

  • OS: MacOS 10.15.3
  • Browser: Chrome 80.0.3987.87
@adamkdean
Copy link
Author

Looks like the author deleted his post. Perhaps the solution here is better messaging to explain that the post/comment thread no longer exists.

@rhymes rhymes added area: publishing experience issues related to an authors experience publishing. Tags, series, etc. tech: fullstack bug always open for contribution labels Mar 12, 2020
@rhymes rhymes changed the title There was an error in your markdown: undefined method `id' for nil:NilClass Error when submitting a comment on a recently deleted post Mar 12, 2020
@rhymes
Copy link
Contributor

rhymes commented Mar 12, 2020

Thanks for the issue! We'll take your request into consideration and follow up if we decide to tackle this issue.

To our amazing contributors: issues labeled type: bug are always up for grabs, but for feature requests, please wait until we add a ready for dev before starting to work on it.

To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please ping @thepracticaldev/oss and we will follow up within 3 business days.

For full info on how to contribute, please check out our contributors guide.

@rhymes
Copy link
Contributor

rhymes commented Mar 12, 2020

Hi @adamkdean, thanks for the issue! I edited the title to add context :)

@cmgorton cmgorton added the external contributors welcome contribution is welcome! label Apr 26, 2021
@cmgorton cmgorton added the bug smash Approved bugs for the DEV community bug smash label May 4, 2021
@saroj990
Copy link
Contributor

saroj990 commented May 5, 2021

Hi @cmgorton, I would like to work on this issue.

@rhymes
Copy link
Contributor

rhymes commented May 5, 2021

@saroj990 I've assigned this to you, thank you 🙌

@cmgorton
Copy link
Contributor

cmgorton commented May 5, 2021

Thanks @saroj990 . If you have any questions feel free to ask here or in your PR once opened. We are here to help along the way.

@saroj990
Copy link
Contributor

saroj990 commented May 6, 2021

What message do we need to display in the alert box for this issue ?

@cmgorton
Copy link
Contributor

cmgorton commented May 6, 2021

Good question. You could add some text for now that indicates the author deleted their comment. Once you have a PR in your reviewers may suggest something else. They will be able to respond more quickly in the PR itself with suggestions 😄

@saroj990
Copy link
Contributor

@cmgorton created a PR for this issue.

@cmgorton
Copy link
Contributor

Hey @saroj990 wanted to check in and see how you are doing with this issue. Did you have any questions or need any help? How are things going with your PR?

@saroj990
Copy link
Contributor

saroj990 commented Jun 1, 2021

Updated the PR with the required changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: publishing experience issues related to an authors experience publishing. Tags, series, etc. bug smash Approved bugs for the DEV community bug smash bug always open for contribution external contributors welcome contribution is welcome!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants