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

Reading: Soft-release reader profile, with framing copy in front #2814

merged 4 commits into from May 8, 2020


Copy link

@kevinrobinson kevinrobinson commented May 7, 2020

Who is this PR for?

primarily for expanding the scope of validation and testing by projects leads (eg, with literacy coaches, reading specialists, classroom teachers)

What problem does this PR fix?

The reader profile could be enabled individually with a feature switch, but we want to open this up a bit more.

What does this PR do?

Because of coronavirus, the release and rollout this will likely happen outside of the PD structures that we'd initially been working towards. So the work on framing the intention and how this connects to problem solving (versus framing centered on "data" or "graphs") will have to get started in a lighter and more organic way.

This adds framing copy up front, with a "click to view" so that folks discovering this on their own get something minimal that hopefully helps with setting the frame and expectations.

Screenshot (if adding a client-side feature)

Screen Shot 2020-05-07 at 10 39 07 AM


Which features or pages does this PR touch?

  • Student Profile

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

  • Included specs for changes
  • Improved specs for existing code in need of better test coverage
  • Manual testing made more sense here

@kevinrobinson kevinrobinson force-pushed the feature/reader-profile-click-to-view branch from 5838800 to e7602e9 Compare May 7, 2020
Copy link
Contributor Author

kevinrobinson commented May 8, 2020


@kevinrobinson kevinrobinson merged commit cc7fe31 into master May 8, 2020
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

1 participant