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

New schema field - group email #1389

Merged
merged 16 commits into from
Feb 14, 2025
Merged

New schema field - group email #1389

merged 16 commits into from
Feb 14, 2025

Conversation

cthoyt
Copy link
Member

@cthoyt cthoyt commented Jan 29, 2025

Related to #1385

  • Add new field to resource model for capturing a group email. This might need a better name than "group email"
  • Add data integrity tests. Initially, we'll only allow curating a group email if there's already an explicit primary contact person. We can consider relaxing this after a pilot and doing a review of relevant curation training materials and generating SOPs for how to communicate this policy to users/contributors
  • add to resource page
  • Pick a better name for this field
  • (optional) update alignment workflows to identify group emails

@cthoyt cthoyt changed the title Add group email field New schema field: group email Jan 30, 2025
@cthoyt cthoyt changed the title New schema field: group email New schema field - group email Jan 30, 2025
@cthoyt cthoyt marked this pull request as ready for review February 14, 2025 07:42
@cthoyt cthoyt merged commit d97de9a into main Feb 14, 2025
12 checks passed
@cthoyt cthoyt deleted the contact-list branch February 14, 2025 07:56
cthoyt added a commit that referenced this pull request Feb 14, 2025
This PR adds a slot for multiple reviews, for situations like in #1406
where multiple people gave meaningful feedback and decided on accepting
a record

This PR also updates the RDF output to add a few things missed with
recent PRs #1389/#1388
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.

1 participant