Enhancement: Shareable Testimonials Auto-Share Settings

massenergize / api

Backend Code for Massenergize Portal. This provides the API to the backend database, and is shared by the various front-end portal projects.

MIT License

5 stars 9 forks source link

Summary / Highlights

This pull request contains enhancements to the testimonial auto-sharing nudge and fixes several bugs. The changes include:

[x] Ensure nudge preparation respects user communication preferences.
[x] Link categories to communities when auto-sharing testimonials, establishing an AND relationship between them.
[x] Fix a few bugs.

Works with Admin PR

Details (Give details about what this PR accomplishes, include any screenshots etc)

Testing Steps (Provide details on how your changes can be tested)

Requirements (place an `x` in each `[ ]`)

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've tested my changes manually.
[x] I've added unit tests to my PR.

Transparency (Project board)

[x] I've given my PR a meaningful title.
[x] I linked this PR to the relevant issue.
[ ] I moved the linked issue from the "Sprint backlog" to "In progress" when I started this.
[ ] I moved the linked issue to "QA Verification" now that it is ready to merge.

Codecov Report

Attention: Patch coverage is 71.86147% with 65 lines in your changes missing coverage. Please review.

Project coverage is 57.54%. Comparing base (0ab19e1) to head (f23ed0c). Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/task_queue/nudges/cadmin_testimonial_nudge.py	19.60%	41 Missing :warning:
src/task_queue/nudges/cadmin_events_nudge.py	69.64%	17 Missing :warning:
src/api/store/testimonial.py	72.72%	6 Missing :warning:
src/task_queue/nudges/user_event_nudge.py	66.66%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #1184 +/- ## ========================================== + Coverage 57.26% 57.54% +0.28% ========================================== Files 488 491 +3 Lines 35963 36050 +87 ========================================== + Hits 20593 20745 +152 + Misses 15370 15305 -65 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

massenergize / api