Skip to content

Filter feedback by object creator#520

Merged
MizukiTemma merged 1 commit intodevelopfrom
feature/filter_feedback_by_content_creator
May 22, 2024
Merged

Filter feedback by object creator#520
MizukiTemma merged 1 commit intodevelopfrom
feature/filter_feedback_by_content_creator

Conversation

@MizukiTemma
Copy link
Copy Markdown
Member

Short description

This PR changes the feedback view so admin users see all the feedback entries but non_admin users only feedback entries related to the objects they created.

Proposed changes

  • add a new query to exclude feedback entries that are not related to the user, if they are not admin.

Resolved issues

Fixes: #353

@MizukiTemma MizukiTemma requested a review from a team April 22, 2024 15:02
@MizukiTemma MizukiTemma force-pushed the feature/filter_feedback_by_content_creator branch from 18cb56c to 931ae4c Compare April 22, 2024 15:20
@MizukiTemma MizukiTemma force-pushed the feature/filter_feedback_by_content_creator branch from 931ae4c to f79a916 Compare April 30, 2024 12:55
Copy link
Copy Markdown
Contributor

@JoeyStk JoeyStk left a comment

Choose a reason for hiding this comment

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

Thank you for your PR :) It looks good and works as expected :) I just have two minor comments, but no blockers :)

Comment thread lunes_cms/cms/feedback_filter.py
Comment thread lunes_cms/core/context_processors.py Outdated
@MizukiTemma MizukiTemma force-pushed the feature/filter_feedback_by_content_creator branch from f79a916 to 5e6b360 Compare May 22, 2024 08:53
@MizukiTemma
Copy link
Copy Markdown
Member Author

@JoeyStk
Thank you for the suggestions 😸 They are applied

@MizukiTemma MizukiTemma force-pushed the feature/filter_feedback_by_content_creator branch from 3e059d0 to b80b762 Compare May 22, 2024 09:20
@MizukiTemma MizukiTemma merged commit 6c6d5ff into develop May 22, 2024
@MizukiTemma MizukiTemma deleted the feature/filter_feedback_by_content_creator branch May 22, 2024 09:24
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.

Filter feedback by creator of related objects

2 participants