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

Rel 3.0.0 #224

Closed
wants to merge 135 commits into from
Closed

Rel 3.0.0 #224

wants to merge 135 commits into from

Conversation

danirus
Copy link
Owner

@danirus danirus commented Oct 10, 2020

Release 3.0.0:

  • Implements enhancement reported in Get rid of frontend dependencies #161.
  • Fixes issue COMMENTS_HIDE_REMOVED seems to not be used (honored) #194, about setting COMMENTS_HIDE_REMOVED and the new setting COMMENTS_XTD_PUBLISH_OR_WITHHOLD_NESTED. Up until v3.0.0 removed comments were listed but their content were not displayed. They showed a "comment has been removed" message instead. That behaviour didn't fully comply with parent's app, django-comments setting COMMENTS_HIDE_REMOVED. By default COMMENTS_HIDE_REMOVED is True, which has the effect of hiding removed comments. As of v3.0.0 this is also the behaviour of django-comments-xtd. Additionally a new setting COMMENTS_XTD_PUBLISH_OR_WITHHOLD_NESTED has been created to control whether nested comments of a comment being removed or approved will be withhold or published.
  • Implements feature reported in Listing top comments #210.

danirus and others added 23 commits August 14, 2020 17:24
 * Watch whether comment model is other than XtdComment.
 * Take into account the SITE_ID.
 * Watch the value of django-comments' COMMENTS_HIDE_REMOVED setting.
@danirus danirus self-assigned this Oct 10, 2020
@danirus danirus marked this pull request as draft October 10, 2020 15:34
@danirus danirus linked an issue Oct 10, 2020 that may be closed by this pull request
danirus and others added 18 commits January 4, 2022 20:18
…templates:

 * django_comments_xtd/templates/comments/form_js.html
 * django_comments_xtd/templates/comments/preview.html
 * django_comments_xtd/templates/comments/reply_button.html
 * django_comments_xtd/templates/comments/reply_form_js.html
just a grammar correction, "envoyé" must be "accordé" (may be in-tune in english) with "commentaire"
and "commentaire" is a male gender noum so only one e at the end of the word.
…reactions:

Use post_delete django signal to trigger deletion of:
 * all nested comments,
 * updated nested_count attributes in parents nodes,
 * delete related CommentReaction and CommentReactionAuthor objects.
The new template tag allows developers to provide their own django template to
render the reactions_panel used by the reactions.js module. The template path
is "comments/reactions_panel_template.html".
@danirus danirus marked this pull request as ready for review February 17, 2022 09:08
This was linked to issues Feb 17, 2022
@danirus danirus closed this Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants