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.

Sign up for GitHub

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

Jump to bottom

Switch push notification lib #346

Merged
merged 38 commits into from
Jul 10, 2018
Merged

Switch push notification lib #346

merged 38 commits into from
Jul 10, 2018

Conversation

diegolmello
Copy link
Member

@RocketChat/ReactNative

Closes #342

diegolmello and others added 30 commits June 13, 2018 09:44
@diegolmello
Changed the way we read RocketChat settings since setting.type won't …
b9b5733 
…be returned from server anymore
@diegolmello
Permissions
a39c859
@diegolmello
Unnecessary action sheet render
e5903ae
@diegolmello
Update gradle
c5f84e1
@diegolmello
Switched testServer to use blob
8e9ea5c
@diegolmello
Merge branch 'settings-fix' into update-sdk-version
ea952b7
@diegolmello
RoomsListHeader search fixed
955a614
@diegolmello
Runs loadMessagesForRoom only if room has at least 20 rows
4c8fb16
@diegolmello
- Logout if user's token expired
c8aab3d 
- Removed update avatar logic
- Profile dialog border on android
@diegolmello
- Animations disabled
32c4193 
- CircleCI set
@diegolmello
Merge branch 'develop' into update-sdk-version
43bc745 
# Conflicts:
#	app/lib/rocketchat.js
#	app/views/RoomView/index.js
#	app/views/RoomsListView/Header/index.js
@diegolmello
Tests updated
0fab2ae
@diegolmello
react-native-navigation installed
7c6e419
@diegolmello
Routes working
bcdf90e
@diegolmello
Tests passing
c9813da
@snyk-bot
fix: package.json & .snyk to reduce vulnerabilities
ce63882 
The following vulnerabilities are fixed with a Snyk patch:
- https://snyk.io/vuln/npm:hoek:20180212

Latest report for rocketchat/rocket.chat.reactnative:
https://snyk.io/test/github/rocketchat/rocket.chat.reactnative
@diegolmello
Setting android
db890f5
@diegolmello
Polyfill
acf68db
@diegolmello
Switched to react-native-navigation v1
6c2d894
@diegolmello
Deep linking
883f421
@diegolmello
Push notifications
cee47dc
@diegolmello
Merge branch 'develop' into navigation
1ead97a 
# Conflicts:
#	android/app/build.gradle
#	app/containers/Routes.js
#	app/views/RoomView/index.js
#	app/views/RoomsListView/Header/index.js
#	app/views/RoomsListView/index.js
#	package-lock.json
@diegolmello
Room search android
e358f82
@diegolmello
Splash
ee00603
@diegolmello
Start app refatored
9704165
@diegolmello
Detox
9224cab
@diegolmello
OAuth now uses react-native-navigator modal
d1497da
@diegolmello
react-native-splash-screen removed
30a349d
@diegolmello
react-navigation removed
7106ab4
@diegolmello
Redux performance
ac79004
@diegolmello diegolmello requested a review from ggazzo July 4, 2018 17:52
@CLAassistant
Copy link

CLAassistant commented Jul 4, 2018 •
edited
Loading

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ diegolmello
❌ snyk-bot
You have signed the CLA already but the status is still pending? Let us recheck it.

@ggazzo ggazzo merged commit 224c421 into develop Jul 10, 2018
@ggazzo ggazzo deleted the push-notification branch July 10, 2018 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ggazzo ggazzo ggazzo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Switch push notification lib
4 participants
@diegolmello @CLAassistant @ggazzo @snyk-bot