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 <Provider> to Code Studio congrats page #18960

Merged
merged 1 commit into from
Nov 7, 2017

Conversation

joshlory
Copy link
Contributor

@joshlory joshlory commented Nov 7, 2017

Sorry! I broke this in PR #18861.

Copy link
Contributor

@Erin007 Erin007 left a comment

Choose a reason for hiding this comment

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

Great! So now I won't have to pass the responsive prop to all the children? But I will need to change all of the children to connect to the store?

@joshlory
Copy link
Contributor Author

joshlory commented Nov 7, 2017

I believe that's correct. I have some followup work to deprecate the new Responsive() class — let's coordinate at standup today to split up the list of components. Thanks!

@Erin007
Copy link
Contributor

Erin007 commented Nov 7, 2017

Oh, wait. The story and test for Congrats probably also need access to the store?

@joshlory
Copy link
Contributor Author

joshlory commented Nov 7, 2017

Yep! I got that in the original PR: https://github.com/code-dot-org/code-dot-org/pull/18861/files#diff-7a973596b91a37da539463d267cd71fcR3

Just forgot to update the actual page as well.

@joshlory joshlory merged commit 5ffcfbc into staging Nov 7, 2017
@joshlory joshlory deleted the add-congrats-redux-provider branch November 7, 2017 21:06
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

2 participants