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 census README #20259

Merged
merged 2 commits into from Jan 27, 2018
Merged

Add census README #20259

merged 2 commits into from Jan 27, 2018

Conversation

drewsamnick
Copy link
Contributor

Adding a bit of documentation about the various kinds of census data. Let me know if you can think of anything I left out or if anything here could use more details.

@drewsamnick
Copy link
Contributor Author

* [Census banner that shows on the studio.code.org teacher homepage](https://github.com/code-dot-org/code-dot-org/blob/staging/apps/src/templates/census2017/CensusTeacherBanner.jsx)

Originally, the Hour of Code and /yourschool census form submissions were written to the Pegasus `forms` table. That data was migrated into `census_submissions` and the mapping between the original `forms` row and the `census_submissions` row is stored in `census_submission_form_maps`. The old Pegasus form handlers are [here](https://github.com/code-dot-org/code-dot-org/blob/staging/pegasus/forms/census.rb) and [here](https://github.com/code-dot-org/code-dot-org/blob/staging/pegasus/forms/hoc_census.rb) and the script used to migrate the data is [here](https://github.com/code-dot-org/code-dot-org/blob/staging/bin/oneoff/move_census_data.rb).

Copy link
Contributor

Choose a reason for hiding this comment

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

Since this is historical, and largely not relevant to consuming the data (now in its new location), I'd suggest moving this to the end and under its own heading, e.g. Background or Prior versions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@Erin007
Copy link
Contributor

Erin007 commented Jan 26, 2018

Yeah for documentation! There's a little bit about the census on the wiki. Do we want to consolidate? Leave the wiki as is? Remove the wiki article since it's very introductory? Reference this README in the wiki? I'm still not 100% clear on what info goes where.

@drewsamnick
Copy link
Contributor Author

I did not realize there was also a wiki page. Since that page is really just a project intro I think it would be best to add a link to this README and otherwise leave it alone. I wanted to keep details here to give a high level overview of the different parts to anyone working on census. This location seems much more discoverable than the wiki.

@drewsamnick drewsamnick merged commit af403a0 into staging Jan 27, 2018
@drewsamnick drewsamnick deleted the census-readme branch January 27, 2018 00:07
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.

None yet

3 participants