Skip to content

React native client #349

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

Closed
wants to merge 3 commits into from
Closed

React native client #349

wants to merge 3 commits into from

Conversation

justin808
Copy link
Member

@justin808 justin808 commented Dec 9, 2016

This change is Reviewable

* added react native
* android sdk update
* basic structure
* added basic router
* better logging
* some styling and navigation
* added api calls
* api posting
* added list view styling
* sorting
* added spinner
* pull to refresh
* button styling
* allowed react rails for http calls
* eslint
* colors
* flow
* android support
* refactored hocs into containers
* added component tests
* sagas first tests
* replaced sagas with thunks
* replaced thunks with effects
* added reducers tests
* added hocs tests
* created common bundle
* added selectors tests
* flow
* moved dispatch out of try
* migrated to local npm
* update for compatibility with thunk
* external lib redux-thunk-effects
* updated name, icon, bundle id, launch screen
* removed outdated code
* ios release
* android app icon + https
* android release
* iOS build 2
* eslint additions
* fixes for review
* returned thunk
* added readme.md
* review comments
* first docs
* images fix
* redux doc
* Selectors docs
Other docs

additions to docs

docs and fixes

minor fix
@justin808 justin808 mentioned this pull request Dec 9, 2016
@justin808
Copy link
Member Author

@alleycat-at-git I will update this PR per our discussions with @robwise and @alexfedoseev. As a useful example to contrast thunks, we might use redux-saga. @robwise, any opinion on that one?

@robwise
Copy link
Contributor

robwise commented Dec 10, 2016

@justin808 Although I like Saga, I thought we had decided to just use plain thunks so it would be easier for people to understand

@justin808 justin808 closed this Jan 11, 2017
@justin808 justin808 deleted the react-native-client branch March 23, 2024 05:59
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.

3 participants