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

Add basic about the data section #28

Merged
merged 6 commits into from
Oct 12, 2023
Merged

Add basic about the data section #28

merged 6 commits into from
Oct 12, 2023

Conversation

jeremiak
Copy link
Collaborator

@jeremiak jeremiak commented Oct 5, 2023

We heard in #22 that journalists were ready to use the data but unclear if it was safe to do so. Here's the start for an about the data section on the homepage to give folks more context about the data to decide if they want to use it.

@netlify
Copy link

netlify bot commented Oct 5, 2023

Deploy Preview for sacramento-campaign-cash ready!

Name Link
🔨 Latest commit c1b539a
🔍 Latest deploy log https://app.netlify.com/sites/sacramento-campaign-cash/deploys/6527b2483fbd88000899ce25
😎 Deploy Preview https://deploy-preview-28--sacramento-campaign-cash.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jeremiak
Copy link
Collaborator Author

jeremiak commented Oct 5, 2023

Hey @nmately and @natebass - what do you think of this? Anything you think we should add or think is missing?

@jeremiak jeremiak marked this pull request as ready for review October 5, 2023 06:02
@natebass natebass self-requested a review October 12, 2023 02:24
natebass and others added 4 commits October 11, 2023 23:26
Remove the browse officials code that was commented out.
Refs #30 because it suggested some of these questions
@natebass
Copy link
Member

natebass commented Oct 12, 2023

The description looks good. I am updating the formatting of the page layout, so I will keep this section in mind. You can merge it now if you want or wait until the page layout is fixed.

#24

@jeremiak jeremiak merged commit d84815b into main Oct 12, 2023
@jeremiak jeremiak deleted the add-about-data branch October 12, 2023 08:48
@rileyschenck
Copy link
Collaborator

Personally I would add more than just "Please let us know" for "What if there's a mistake?" Like, "The only data that has been modified are the slight naming variations to assist with aggregating data by name. If you notice an inconsistent name, or any other problem or mistake that you think may exist either in the source data or our aggregation of it, please let us know"

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.

3 participants