Refactor the SearchContext

FilipMessa commented 5 years ago

Summary

Hydrate the SearchContext from the URL at context creation
Removed the setStateFromQueryParams from components componentDidMount, because it shouldn't be necessary.
Moved the BookingToken from BookingContext to SearchContext, for code simplicity (need to rethink if the BookingContext is necessary)

Plan for next PRs

Move the SearchContexts parser methods to helper file, and write tests.
Render empty PassengerCards based on count of passengers from SearchContex, similar behavior like on kiwi.com

netlify[bot] commented 5 years ago

Storybook from packages/universal-components

Built with commit 73d7672fda4392f926869faf3903c386743df0ea

FilipMessa commented 5 years ago

@aalises

LGTM! The setStateFromQueryParams is not necessary/not used anymore, right? Could we remove it or will be needed somewhere else?

Good point, I guess we could remove it. I will do that. Thanks

FilipMessa commented 5 years ago

@aalises done

kiwicom / margarita