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

comments_async: fix some issues with child comments #1642

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

goapunk
Copy link
Contributor

@goapunk goapunk commented Jun 20, 2024

  • fix child comments not being editable by creator
  • hide comment form headline when editing

fixes #liqd/adhocracy-plus#2716

Tasks

  • PR name contains story or task reference
  • Documentation (docs and inline)
  • Tests (including n+1 and django_assert_num_queries where applicable)
  • Changelog

- fix child comments not being editable by creator
- hide comment form headline when editing
@goapunk goapunk requested review from m4ra and hom3mad3 June 20, 2024 09:31
@@ -421,6 +422,8 @@ export default class Comment extends React.Component {
comments={this.props.child_comments}
anchoredCommentId={this.props.anchoredCommentId}
anchoredCommentParentId={this.props.anchoredCommentParentId}
hasCommentingPermission={this.props.hasCommentingPermission}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we forgot to pass in the permissions for the child comments to allow editing

@hom3mad3 hom3mad3 merged commit 811c326 into main Jun 20, 2024
3 checks passed
@hom3mad3 hom3mad3 deleted the jd-2024-06-child-comment-issues branch June 20, 2024 10:09
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.

2 participants