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

[DOCS] Add native mobile app links into README and update button images #7909

Merged
merged 3 commits into from
Aug 30, 2017

Conversation

rafaelks
Copy link
Contributor

@RocketChat/core

This PR changes the link to download the mobile apps on README file, since our native apps are now much more stable and being used by many people on the community.

@JSzaszvari
Copy link
Contributor

can we leave the old links there as well. as nice as the native apps are they don't even come close to the functionality offered by the cordova apps.

@JSzaszvari
Copy link
Contributor

eg just checked the android app, links not being rendered correctly, no admin settings, no user preferences, no message actions,. attachment messages not rendering correctly

I don't think it's time to completely remove the links for the old apps.

@graywolf336
Copy link
Contributor

I'm going to agree with @JSzaszvari here, the native applications just aren't on par with the cordova version at the moment and until I can use multiple servers on the native applications it is useless to me.

@rafaelks
Copy link
Contributor Author

Thanks for your feedback @graywolf336 and @JSzaszvari! I've updated the file with Hybrid mentions and brand new icons, matching the new stores designs.

@rafaelks rafaelks changed the title [DOCS] Change Mobile apps links to the native ones [DOCS] Add native mobile app links into README and update button images Aug 28, 2017
@rodrigok rodrigok added this to the 0.59.0-rc.3 milestone Aug 30, 2017
@rodrigok rodrigok merged commit 1b651bf into RocketChat:develop Aug 30, 2017
rodrigok added a commit that referenced this pull request Aug 30, 2017
[DOCS] Add native mobile app links into README and update button images
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.

4 participants