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

Feature/export for www #1425

Conversation

chrisgarrity
Copy link
Contributor

Resolves

FIxes #1336.

Proposed Changes

  • Refactor GUI components to expect state to be in state.gui
  • refactor reducers to have intl and ScratchPaint separate
  • move the current GUI to playground folder
  • add index.js that exports the GUI as default, GuiReducer and ScratchPaintReducer

Reason for Changes

Allows the GUI to be imported for use in scratch-www

Test Coverage

Current tests continue to run

Separate the GUI and scratchPaint reducers, and export both for combining by wrapping component.

Change examples to import GUI from new index.js instead of from container.
Looks like something should happen with all the SVG and PNG outputs.
Question - do all of the modules need the same assets copies. If so can the plugins be part of the base? Apparently sometimes plugin order matters, and other times it doesn’t.
Move devServer into base to correctly set the port.
Remove unneeded node version.
@chrisgarrity chrisgarrity added this to the February 2018 milestone Feb 8, 2018
@chrisgarrity chrisgarrity requested a review from rschamp February 8, 2018 23:14
@rschamp rschamp removed their request for review February 15, 2018 16:50
@rschamp
Copy link
Contributor

rschamp commented Feb 28, 2018

We've pretty much settled on not doing things this way, so closing this one.

@rschamp rschamp closed this Feb 28, 2018
@chrisgarrity chrisgarrity deleted the feature/export-for-www branch April 23, 2018 14:08
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.

2 participants