Skip to content

Reading: Add validations for different assessments, not yet applied#2621

Merged
kevinrobinson merged 4 commits into
masterfrom
patch/reading-validations
Sep 24, 2019
Merged

Reading: Add validations for different assessments, not yet applied#2621
kevinrobinson merged 4 commits into
masterfrom
patch/reading-validations

Conversation

@kevinrobinson

@kevinrobinson kevinrobinson commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

Adds a validation function for reading data points, not yet applied to the model. This is part of data quality checks on fall assessment data.

It also adds a transformation function in the processor, specifically to handle how importing data produced by formulas works.

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

  • Included specs for changes
  • Manual testing made more sense here

@kevinrobinson

Copy link
Copy Markdown
Contributor Author

selfie

@kevinrobinson kevinrobinson merged commit 6cd8842 into master Sep 24, 2019
@kevinrobinson kevinrobinson deleted the patch/reading-validations branch September 24, 2019 18:56
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.

1 participant