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 M-O) #8378

Draft
wants to merge 2 commits into
base: nl-remove-author-data-k-to-l
Choose a base branch
from

Conversation

nick-mon1
Copy link
Contributor

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

Summary

Metadata removal for author with M to O.

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-k-to-l February 5, 2025 18:58
@@ -7,32 +7,17 @@ display_name: "Mark Anthony Dingbaum"
first_name: "Mark Anthony"
last_name: "DIngbaum"
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: "DIngbaum"
last_name: "Dingbaum"

@@ -7,32 +7,17 @@ display_name: "Mary King"
first_name: "Mary King"
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
first_name: "Mary King"
first_name: "Mary"

@@ -7,32 +7,17 @@ display_name: "Mary King"
first_name: "Mary King"
last_name: "Mary King"
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: "Mary King"
last_name: "King"

@@ -8,26 +8,14 @@ display_name: "Nancy Kreidler"
first_name: "Nancy Kreidler "
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
first_name: "Nancy Kreidler "
first_name: "Nancy"

@@ -8,26 +8,14 @@ display_name: "Nicolas Pjontek"
first_name: "Nicolas "
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
first_name: "Nicolas "
first_name: "Nicolas"

Copy link
Contributor

Choose a reason for hiding this comment

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

flagging unusual use case: full name doesn't match display name

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