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

[ENG-6682] Add CCing and Reply-to features to Institutional Access User Messaging #240

Conversation

Johnetordoff
Copy link
Owner

@Johnetordoff Johnetordoff commented Dec 10, 2024

Purpose

Add the CCing and Reply-to features that were added as a result of the ARB.

Changes

  • adds bool fields to serializers to indacte cc and/or reply-to
  • cleans-up logic in email utilities and updates them to allow for ccing and reply-to
  • adds tests for the new bhavior.

QA Notes

Please make verification statements inspired by your code and what your code touches.

  • Verify
  • Verify

What are the areas of risk?

Any concerns/considerations/questions that development raised?

Documentation

Side Effects

Ticket

PR:

I decided to split this off this PR, because it was only for features discussed in the ARB.

Johnetordoff and others added 16 commits November 15, 2024 13:35
* add management command to migrate preprint affiliations

* make sure migrations uses primary instead of replica, improve tests

* remove redundant permission

---------

Co-authored-by: John Tordoff <>
…penScience#10783)

* Add OOPSpam and Akismet metrics to spam report

* Add spam/ham metrics for OOPSpam and Akismet with FK join fix, test cleanup

* Add private spam metrics report with preprint inclusion

* Validate category; add PrivateSpamMetricsReporter to monthly reports

---------

Co-authored-by: Uditi Mehta <uditimehta@COSs-MBP.lan>
prevent counted-usages with different `action_labels` from overwriting
each other -- deduplicate only when `action_labels` match exactly
…-and-i-with-dashboard-update

[ENG-6685] B&I Release Fixes
B&I Release

- Migrate Preprint Affilations
- Add OOPSpam and Akismet metrics to spam report
- Add PrivateSpamMetricsReport
- Update PrivateSpamMetricsReporter to work with refactored MonthlyReporter
- Fix duplicate reports when run for past years
- Fix counted-usage clobbers
@Johnetordoff Johnetordoff force-pushed the institutional-access-user-message-arb branch from d149430 to 25d7857 Compare December 10, 2024 15:00
Johnetordoff and others added 3 commits December 11, 2024 10:01
…tutional-access-user-message

[ENG-6682] Add UserMessage feature for Institutional Access
…terForOpenScience/osf.io into institutional-access-user-message-arb

* 'feature/institutional_access' of https://github.com/CenterForOpenScience/osf.io:
  add user message read/write permissions to full
  add new user message oauth scope and throttling classes
  Fix backfill, report
  Update changelog and bump versions
  Follow-up fix for target/next (start/end) month
  Fix failures caused by base class MonthlyReporter update
  [ENG-6506] Fix: counted-usage clobbers (CenterForOpenScience#10799)
  [ENG-6435] Fix: duplicate reports when run for past years (CenterForOpenScience#10800)
  Add PrivateSpamMetricsReport (CenterForOpenScience#10791)
  [ENG-4438] Add OOPSpam and Akismet metrics to spam report (CenterForOpenScience#10783)
  [ENG-6364] Migrate Preprint Affilations (CenterForOpenScience#10787)
@Johnetordoff Johnetordoff force-pushed the institutional-access-user-message-arb branch 3 times, most recently from 87711e9 to 2130551 Compare December 12, 2024 13:50
@Johnetordoff Johnetordoff force-pushed the institutional-access-user-message-arb branch from 2130551 to 8de394a Compare December 12, 2024 13:55
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.

5 participants