Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add Hive iOS #523

Merged
merged 1 commit into from Aug 22, 2014

Conversation

Projects
None yet
3 participants
Contributor

javgh commented Aug 14, 2014

Another pull request from the Hive people. ;-)

This one adds Hive iOS. Hive iOS is basically just Hive Web wrapped with Cordova to turn it into an app. So code-wise there isn't much new, although the design has been slightly adjusted to better match iOS. I hope this can be added to the iOS mobile category. Thanks!

Contributor

schildbach commented Aug 14, 2014

If the web page isn't open source, it should be
transparency: "checkfailtransparencyremote"

Contributor

saivann commented Aug 14, 2014

@javgh Could you use a single description both for Hive iOS and Hive Web in en.yml? Translating takes a lot of effort so this would be appreciated.

Under mobile:, can you use the Android app instead? Just for consistency with other wallets, which all show their Android app by default for the "Mobile" category.

I was also wondering if we should be using the checkfailtransparencynew score for Hive Web, Hive iOS and other wallets to be added in the following weeks, since these projects are relatively new.

Contributor

saivann commented Aug 14, 2014

@schildbach You probably mean "If the web page is loaded remotely"?

Contributor

javgh commented Aug 14, 2014

Ok, collapsed the descriptions to a single one for Hive Web and Hive iOS.

Also displayed Android as the default for mobile: . I'm not sure if that default view should list both Android and iOS platform as supported. It seems that otherwise you wouldn't necessarily discover, that there is also an iOS Hive product, but then again, it's somewhat different from Hive Android, so showing the iOS symbol next to a Hive Android screenshot seems confusing. So now the default view only shows Android as platform and then hopefully visitors will specifically filter down to iOS.

Hive iOS does not load any code from a remote server, the HTML and Javascript is already packaged and comes with the app. So I don't think that checkfailtransparencyremote applies.

Contributor

saivann commented Aug 14, 2014

If the code is part of the app and not loaded remotely, that's indeed how I see it too.

Contributor

schildbach commented Aug 14, 2014

@javgh Ah ok, didn't know that. So it only runs locally and all of the app code, html and javascript is open source (and versioned in your git repo)?

Contributor

javgh commented Aug 15, 2014

@schildbach That's right. The Hive Web source code can be found in this repository and the Hive iOS repository is a fork of that with Cordova build instructions added and some iOS tweaks applied and is located here: https://github.com/hivewallet/hive-ios . Both projects use a GPL license.

Contributor

saivann commented Aug 19, 2014

LGTM. In the absence of critical feedback, this pull request will be merged on August 21th.

@saivann saivann merged commit 908e60d into bitcoin-dot-org:master Aug 22, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment