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.

Sign up for GitHub

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

Jump to bottom

Remove author landing page and metadata (authors T-Z) #8381

Open
wants to merge 5 commits into
base: nl-remove-author-data-p-to-s
Choose a base branch
from
Open

Remove author landing page and metadata (authors T-Z) #8381

wants to merge 5 commits into from

Conversation

nick-mon1
Copy link
Contributor

@nick-mon1 nick-mon1 commented Feb 6, 2025 •
edited
Loading

Summary

Metadata removal for authors T-Z.

Note

Will be merged into #8216 and also includes author removal for authors with no published content.

Solution

Removes all other author fields.

Fields to Keep

  • display_name
  • first_name
  • last_name
  • agency_full_name
  • github (github username)
  • profile_source (digit-light, digit-dark, blank)

How To Test

  1. Only fields listed above are displayed
  2. Remove non-published authors

@nick-mon1 nick-mon1 self-assigned this Feb 6, 2025
@nick-mon1 nick-mon1 changed the base branch from main to nl-remove-author-data-p-to-s February 6, 2025 16:49
@nick-mon1 nick-mon1 mentioned this pull request Feb 6, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bonnieAcameron bonnieAcameron bonnieAcameron approved these changes

@ToniBonittoGSA ToniBonittoGSA Awaiting requested review from ToniBonittoGSA

Assignees

@nick-mon1 nick-mon1

Labels
Content: Authors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nick-mon1 @bonnieAcameron