Skip to content
This repository has been archived by the owner on Nov 1, 2018. It is now read-only.

Rewrite themes and others with typescript #48

Merged
merged 125 commits into from
Oct 31, 2017
Merged

Rewrite themes and others with typescript #48

merged 125 commits into from
Oct 31, 2017

Conversation

lucat1
Copy link
Collaborator

@lucat1 lucat1 commented Oct 13, 2017

Closes #24
Closes #29
Closes #40
Closes #44
Closes #43 (With SSR)

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 13, 2017

Removed netlify checks.

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 13, 2017

🔥 WARNING TO SELF 🔥

  • Theme prop is not yet correctly passed to the renderer template string
  • Support and test SRR(should be easy)

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 13, 2017

Really interested in implementing siddharthkp/bundlesize with this since bundle size and speed are one of our major concerns

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 27, 2017

Dropped #23 as an objective since it was going to take way too long. We'll cover it in the next PR along with a proper rewriting and not just conversion to TS

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 31, 2017

Right now i'm struggling to bundle these packages with a .babelrc. Hope to fix it soon so this can be merged! 🎉

@lucat1
Copy link
Collaborator Author

lucat1 commented Oct 31, 2017

This PR is finally over! 🎉

@lucat1 lucat1 merged commit bff2f2d into master Oct 31, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants