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

Remove author landing page and metadata (authors D-E) #8374

Open
wants to merge 6 commits into
base: nl-remove-author-data-b-to-c
Choose a base branch
from

Conversation

nick-mon1
Copy link
Contributor

@nick-mon1 nick-mon1 commented Feb 3, 2025

Summary

Metadata removal for author ranges D-E.

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 changed the base branch from main to nl-remove-author-data-b-to-c February 3, 2025 18:51
Copy link

github-actions bot commented Feb 3, 2025

🔍 Preview in Federalist

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No agency for this one. That ok?

Copy link
Contributor

@bonnieAcameron bonnieAcameron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, pending the following: added small suggestions in the metadata, mostly typos, one missing agency.

@@ -3,10 +3,7 @@ display_name: Elham Tabassi
first_name: Elham
last_name: Elham
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
last_name: Elham
last_name: Tabassi


# Where can people learn more about your agency or program? Provide a full URL [e.g. 'https://www.example.gov/']
bio_url: ""

# Agency Full Name [e.g. U.S. General Services Administration]
agency_full_name: "General Services Administration"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
agency_full_name: "General Services Administration"
agency_full_name: "U.S. General Services Administration"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants