Skip to content

Rename visibleInWhois fields to visibleInRdap#2863

Merged
gbrodman merged 1 commit into
google:masterfrom
gbrodman:visibleInRdap
Nov 4, 2025
Merged

Rename visibleInWhois fields to visibleInRdap#2863
gbrodman merged 1 commit into
google:masterfrom
gbrodman:visibleInRdap

Conversation

@gbrodman

@gbrodman gbrodman commented Oct 27, 2025

Copy link
Copy Markdown
Collaborator

Still part of b/454947209, removing references to WHOIS where we can. We keep the registrar type and the column names (at least for now) because changing those is much more complicated.


This change is Reviewable

@gbrodman
gbrodman force-pushed the visibleInRdap branch 3 times, most recently from e534c93 to 506f15b Compare October 28, 2025 18:51
@gbrodman
gbrodman requested a review from ptkach October 28, 2025 20:54
@gbrodman
gbrodman force-pushed the visibleInRdap branch 2 times, most recently from 7ce8e74 to 1e1b70e Compare October 29, 2025 20:06
Still part of b/454947209, removing references to WHOIS where we can. We
keep the registrar type and the column names (at least for now) because
changing those is much more complicated.
@gbrodman

gbrodman commented Nov 3, 2025

Copy link
Copy Markdown
Collaborator Author

friendly ping

@ptkach ptkach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ptkach reviewed 28 of 28 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @gbrodman)

@gbrodman
gbrodman added this pull request to the merge queue Nov 4, 2025
Merged via the queue into google:master with commit dee132d Nov 4, 2025
9 checks passed
@gbrodman
gbrodman deleted the visibleInRdap branch November 4, 2025 18:41
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