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

[Feature Request] Publish the android app #6

Closed
MohamedBassem opened this issue Mar 26, 2024 · 46 comments
Closed

[Feature Request] Publish the android app #6

MohamedBassem opened this issue Mar 26, 2024 · 46 comments
Labels
feature request New feature or request

Comments

@MohamedBassem
Copy link
Collaborator

MohamedBassem commented Mar 26, 2024

The mobile app is already cross-platform. We just need to test and publish it to the play store.

EDIT2:

The app is now live on the play store: https://play.google.com/store/apps/details?id=app.hoarder.hoardermobile&pcampaignid=web_share

EDIT:

Ok, so google just accepting the app for closed testing!

To join the closed testing group, you'll need to join this google group: https://groups.google.com/g/hoarder-android-testers

Once you've joined (I guess you might need to give it a minute or two), you can use the following link to install the app from the app store:

We will need at least 20 testers, for 2 weeks for the app to get listed on the play store. Please give it a try and let me know how it goes!

@MohamedBassem MohamedBassem added the feature request New feature or request label Mar 26, 2024
@runinit
Copy link

runinit commented Mar 27, 2024

I can test this if needed

@MohamedBassem
Copy link
Collaborator Author

As of (26b53e2), the android app is ready for submission to the play store. Going through the submission process right now :)

@MohamedBassem
Copy link
Collaborator Author

Ok, the app right now is under google's review. Will share the testing link once it's available. We'll need to get some testers before it's widely distributed in the play store.

@DavidN0809
Copy link

I am open to testing as well

@mp-strachan
Copy link

Also happy to opt in for testing

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

If the .apk was linked in this thread, i'd put it on a phone I have...
[Read: please also enroll me in testing.]

@MohamedBassem
Copy link
Collaborator Author

Here's the apk: https://drive.google.com/file/d/1VEeN6KDCHe0jy8PmX0I3sZLtY5DZ2Wd3/view?usp=sharing (first time doing this, so hope I've done it right).

Will share the testing invitation link once google allows me to do so.

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

Network request failed
Server Address
192.168.1.2:3004
Email
iconoclasthero@anemailsite.com
Password
********

Also, the following happens:

  • when I start typing in the password, gboard defaults to a capital letter, please make it not do that.⸸
  • http://192.168.1.2:3004 failed for me the first time on some sort of error but worked this time
  • when trying to edit the server address, the text is entered backwards, so adding the http:// to 192.168.1.2 results in `//:ptth192.168.1.2
  • you should have some sort of verification on the email address to the point that it is in the form of user@*domain.tld so users don't enter a username instead of a password like i did..you know to idiot proof the login.
  • I noticed this on audiobookshelf: be careful that you don't let the entry include a [white]space after the email address/password. messes me up on abs all the time.

⸸I do not expect password entry to default to caps (e.g., the way it does for text entry) and i'm sure others will get tripped up there as well. don't know if it would happen on other keyboards besides gboard but i assume that one is pretty popular.

@MohamedBassem
Copy link
Collaborator Author

You probably should add the protocol as well, so http://192.168.1.2:3004 (or https if you're using https)

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

Settings > Log > Logs me out instead of showing log.
This phone has android 8 on it if that makes a difference.

"Home" is truncated to "Hom" at the top of the screen

@MohamedBassem
Copy link
Collaborator Author

@iconoclasthero thanks a lot for the feedback :) You're the first android tester, so expect some bugs :) Please keep the bug report coming.

  1. I'll look into the weird behavior of the server address. I think I know what's happening.
  2. I'll look into the auto captilize thing as well.
  3. Settings > Log. There's no Log it's actually a Log out button :D
Screenshot 2024-03-30 at 3 27 54 PM

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

I call bullshit! I found the Log button!
ok, you saw the image, i don't feel like making it a reasonable size and dealing with imgur. FFS is this a PITA!

@MohamedBassem
Copy link
Collaborator Author

wtf, how did this happen. Mobile development is so weird 😂

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

On the plus side, here's how I shared that image: I took a screenshot inside the android app, went to share the image from the screenshot dialog, and the button f/hoarder came up...which i hit. I went to the pc, had to hit refresh, and it was only a white background with no information, but it was there.

Screenshot from 2024-03-30 11-51-12

@MohamedBassem
Copy link
Collaborator Author

MohamedBassem commented Mar 30, 2024

You having to hit refresh is tracked in #29.

It being white, I think have something to do with how I "stretch" the content in the container and the screenshot being mostly white. I think if you click the maximize button, you'll see the full screenshot.

Typically, there would be tags, but judging from your other bookmarks, it seems that you don't have automatic tagging enabled. That's why the card feels so "empty" 😅 I'm open to suggestions on how to make it more better looking.

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

Well, this is bluesky, but i'd want "something" to look at the image and find the "most interesting" part of it and feature that. I.e., in this case, that would clearly be the top of the image, not the bottom. How you do that, not sure. I think it's possible to set global options for how images are displayed, i.e., are they thumbed and centered, stretched, ... and then as this develops, that should change to being able to individually set how those images are shown.

@MohamedBassem
Copy link
Collaborator Author

Yeah, better rendering for images is on my todo list (and my todolist is getting quite big) :)

@JFPCreations
Copy link

Thanks for the app. So far so good. No problem found.

@Reledia
Copy link

Reledia commented Mar 30, 2024

I'm trying the app on android and I can find no issues with it. I would only suggest to implement a re-fetch button in the note menu for when inference fails. (And possibly even a tag/error that notify of this)

@iconoclasthero
Copy link

iconoclasthero commented Mar 30, 2024

Typically, there would be tags, but judging from your other bookmarks, it seems that you don't have automatic tagging enabled. That's why the card feels so "empty" 😅 I'm open to suggestions on how to make it more better looking.

I wanted to ask about that...is there a discord or anything?

I see that there's a couple API tokens in there. Not sure if they're mine for OpenAI and I don't see where I can enable automatic tagging.

Screenshot from 2024-03-30 13-02-34

@MohamedBassem
Copy link
Collaborator Author

I've created a discord channel: https://discord.gg/NrgeYywsFh

Let's chat there.

@lucius346346
Copy link

lucius346346 commented Mar 31, 2024

Network request failed
Server Address
192.168.1.2:3004
Email
iconoclasthero@anemailsite.com
Password
********

I'm having the same error but it persist. Also same error on Firefox addon (NetworkError when attempting to fetch resource). Is it some misconfiguration on my part? I've tried it with or without http://. The web version works fine though.

@MohamedBassem
Copy link
Collaborator Author

@lucius346346 check #23 for the firefox problems.

As for the app, what errors are you getting?

@lucius346346
Copy link

@lucius346346 check #23 for the firefox problems.

As for the app, what errors are you getting?

It's "Network request failed" no matter what I type, including blank. I'm sure I'm typing the right credentials - I can log with them via browser.

@MikeKMiller
Copy link

Same here with the android app.

@stanrc85
Copy link

stanrc85 commented Apr 1, 2024

Same here, tried both with ip and hostname

@MohamedBassem
Copy link
Collaborator Author

MohamedBassem commented Apr 1, 2024

@lucius346346 @MikeKMiller @stanrc85 any chance any of you is using a self-signed (or invalid) https cert?
Btw, if any of you would like to hope onto Hoarder's discord so that we can debug it together, I'd be thankful.

@stanrc85
Copy link

stanrc85 commented Apr 1, 2024

No https for me, I posted in the discord!

@MikeKMiller
Copy link

I have no https either

@lucius346346
Copy link

Also, no.

@MohamedBassem
Copy link
Collaborator Author

I managed to repro locally, and the good news is that I think I found the problem. Apparently android release builds by default refuse to connect to HTTP endpoinds (requiring https basically). This behavior is different in release builds vs development builds and hence why I didn't catch it during development.

I'll try some of the mitigations I googled and report back.

@MohamedBassem
Copy link
Collaborator Author

I checked immich for example and it seems to be setting this usesCleartextTraffic=true. That I hope will be the fix. Trying it now.

https://github.com/immich-app/immich/blob/861b72ef0493bb6272c1a1b16e5cdee6792fb4b6/mobile/android/app/src/main/AndroidManifest.xml#L3

@MikeKMiller
Copy link

I was just going to suggest that.

@MohamedBassem
Copy link
Collaborator Author

Good news, it worked! At least on my test phone!

I've updated the APK link with the new version:

https://drive.google.com/file/d/1VEeN6KDCHe0jy8PmX0I3sZLtY5DZ2Wd3/view?usp=sharing

Please give it a try and let me know :)

@MikeKMiller
Copy link

That worked, I can login now.

@stanrc85
Copy link

stanrc85 commented Apr 1, 2024

Success!

@MohamedBassem
Copy link
Collaborator Author

Ok, so google just accepting the app for closed testing!

To join the closed testing group, you'll need to join this google group: https://groups.google.com/g/hoarder-android-testers

Once you've joined (I guess you might need to give it a minute or two), you can use the following link to install the app from the app store:

We will need at least 20 testers, for 2 weeks for the app to get listed on the play store. Please give it a try and let me know how it goes!

@woaloo
Copy link

woaloo commented Apr 10, 2024

Hi,
I've just install this app with my self-hosted Hoarder instance. Woaw, it's working great !
Thanks a lot for your job
image

@Batman313v
Copy link

I didn't see any conversations in the google group so I'll just mention it here, Please add dark mode. I installed and opened the app (it's midnight local time) and burned my bookmarks to my retinas 😂 Other than that, it'd be nice to have one of the following options:

Multiple "Share" options; a quick one (how it is now) and one that pulls up a menu to add tags.

A setting to enable a "complex" share menu with that feature 😁

(Please note: not complaining just giving feedback as a early tester. Love the project, great app 👍)

@MohamedBassem
Copy link
Collaborator Author

@Batman313v I've just landed dark mode in c46482c ! I'll cut a new mobile release now and it should be available on your phone when google approves the new release!

As for the complex share menu, this is planned.

@MohamedBassem
Copy link
Collaborator Author

Dark mode show now be available in version 1.4.0.

As for publishing the app to the play store. According to google, we're at day 9 of 20. 11 more days to go before we're able to publish the app directly to the play store!

@mycodex-dev
Copy link

Just installed the Android app and entering the server address on the login screen has a behavior where Gboard hides after each key press

h, keyboard hides, t, keyboard hides, t, keyboard hides, p, keyboard hides...

I had to type the server address in another app and copy/paste into the server address field in your app because it would have taken too long.

@MohamedBassem
Copy link
Collaborator Author

MohamedBassem commented Apr 19, 2024

@mycodex-dev I've just now released v1.5 for the mobile app (might take an hour until google propagates it), that fixes this particular issue. Sorry for the hassle.

EDIT: Commit for reference: 4f9b6c8

@mycodex-dev
Copy link

@mycodex-dev I've just now released v1.5 for the mobile app (might take an hour until google propagates it), that fixes this particular issue. Sorry for the hassle.

EDIT: Commit for reference: 4f9b6c8

Awesome! Love the mobile app so far. This completely removed my need for Pocket and even Google Keep with the notes functionality with markdown support chef's kiss.

Keep up the great work!

@MohamedBassem
Copy link
Collaborator Author

Hey folks, thanks a lot for your support! Hoarder's android app is now live on the play store!. If you're interested in more stability, you can now switch to the prod app, if you're interested for the latest updates, you can stay on the testing list. And while, you're at it, maybe maybe leave a review on the play store ;)
Thanks!

@scubanarc
Copy link

Android app works perfectly! I really like having it as a share target. Left a glowing review :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests