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

On the road to simpler components & better dev experience #14

Closed
wants to merge 15 commits into from

Conversation

xavxyz
Copy link
Member

@xavxyz xavxyz commented Apr 5, 2017

🚧 ➡️ 🛣 work in progress pull request

State of the art, summer 2016 🚀

The project is developed in 24 hours: the React components are working, though it takes time to read through them / make a modification (ex: jQuery was actually used somewhere, at some point, to clean :empty DOM 🛩 🌮)

Pimped components ✨

This PR intends to:

  • provide a better dev experience thanks to a component explorer (Storybook) and tests (snapshot testing with Jest);
  • components easier to read, manage and update by tending toward functional components (recompose ftw) and using of CSS-in-JS (styled-components).

What's going on now? 🛠

  • Setup Storybook with mocked data apollo-test-utils
    • it's working!
    • uncertainty: are there better way to mock data than the way I did?
  • Setup snapshots testing thanks to Storyshots
  • And of course, write useful stories helping understand the app and refactor the components
    • TourBadge is used in RecentTours & Tour 🤓
    • still a lot to do there 👍

Note: I've hit (and solved/got around) quiet a few walls while setting up the tooling, happy to talk about the decision taken. And happy to talk about the components refactoring / styling / direction of the project 🙂

@xavxyz
Copy link
Member Author

xavxyz commented Jul 1, 2017

I'm closing this, I'm working on something really simpler (lol) and with a better user experience

Sneak peek:

screencast

@xavxyz xavxyz closed this Jul 1, 2017
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