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

Update first_party_sets.JSON to add acbj.com FPS #19

Closed
wants to merge 3 commits into from

Conversation

brownwolf1355
Copy link

American Cities Business Journals FPS

American Cities Business Journals FPS
Added links to the definitions of set primary and set member in the section "Set-level technical validation" to clarify the meaning of member and primary in this context.
@cfredric
Copy link
Collaborator

cfredric commented May 2, 2023

Hi Ralph - did you mean to associate 2544a8d with this PR? I think we should keep substantive changes separate from editorial changes, so we should submit 2544a8d as its own PR.

@brownwolf1355
Copy link
Author

brownwolf1355 commented May 2, 2023 via email

Clarifying the language in Set-level technical validation bullet to link to definitions of set primary and set member in the definitions section.
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