Skip to content

Upgrades to contact messages (v2)#5367

Open
joemull wants to merge 23 commits into
r-v1.9.xfrom
b-4138-contact-message-upgrades
Open

Upgrades to contact messages (v2)#5367
joemull wants to merge 23 commits into
r-v1.9.xfrom
b-4138-contact-message-upgrades

Conversation

@joemull

@joemull joemull commented Jun 25, 2026

Copy link
Copy Markdown
Member

Replacement PR for #4845 which was merged and then reverted.


Closes #4138.
Closes #1720.

Notes

  • Closes the linked issues
  • Connects the contact person and contact message models to account, so that we do not have names and emails outside of the Account object
  • Renames the contact person and contact message models to make them easier to read and work with
  • Removes stored client IPs to comply with GDPR
  • Adds the Django auto-documentation views, which are helpful when searching for template tags and filters, and which show you view and url pairs
  • Documents a few things
  • Tests for the contact manager and the press and journal contact pages

Screenshots for contact system

Screenshot from 2025-07-21 17-32-46 Screenshot from 2025-07-21 17-33-10 Screenshot from 2025-07-21 17-33-20 Screenshot from 2025-07-21 17-33-46 Screenshot from 2025-07-21 17-34-03 Screenshot from 2025-07-21 17-34-14

Screenshots for admin docs feature

I enabled this in the process of doing this work. It is enormously helpful to view a comprehensive list of things in Janeway sometimes.

Screenshot from 2025-07-21 17-35-02 Screenshot from 2025-07-21 17-35-18 Screenshot from 2025-07-21 17-40-34

@joemull joemull marked this pull request as draft June 25, 2026 09:30
@joemull joemull changed the title Upgrades to contact messages Upgrades to contact messages (v2) Jun 25, 2026
@joemull joemull requested a review from mauromsl June 25, 2026 12:17
@joemull joemull marked this pull request as ready for review June 25, 2026 12:21
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