WildAid / o-fish-ios

iOS app for the Officer's Fishery Information Sharing Hub (O-FISH). The mobile app allows fisheries officers to document and share critical information gathered during a routine vessel inspection.
Apache License 2.0
33 stars 15 forks source link

User avatar photo doesn't display when logging into app #289

Open am-MongoDB opened 4 years ago

am-MongoDB commented 4 years ago

When first logging into the app, the user is taken to the map screen, but their avatar image isn't displayed until they navigate to another view and then return

o-fish-wildaid commented 4 years ago

Raise new issues resulting from using/testing the apps

pankova commented 3 years ago

Hello! 🙂 Could I help you with it as Hacktoberfest participating?

am-MongoDB commented 3 years ago

Hi @pankova I've assigned this issue to you – welcome to the project!

pankova commented 3 years ago

@am-MongoDB hello! Could you say:

1) Should I use AWS-secret-key for step 5.1.d? If yes, where I could get it? 2) On step 6.6 after command

realm-cli import --strategy=replace --include-dependencies --app-id=REALM_APP_ID

I get a log.

Import dependencies
Please confirm the changes shown above: [y/n]: y
Creating draft for app...
Draft created successfully...
Importing app...
Deploying app...
Deploying app...
Done.
transpiling dependencies started.
exec: "transpiler": executable file not found in $PATH
Sheeri commented 3 years ago

Hi @pankova - how did you install Realm CLI? and what version of realm CLI do you have?

pankova commented 3 years ago

Hi @pankova - how did you install Realm CLI? and what version of realm CLI do you have?

Hi) 1.1.0

Shubham0812 commented 3 years ago

@am-MongoDB is this done? I'd love to take this up 👍

pankova commented 3 years ago

Just in case, it isn't a critical problem. I have built the app. I will do the task after work 🙂

am-MongoDB commented 3 years ago

Hi @pankova the issue is still open, and so we'd love to receive a PR. For the transpiler warning, you can probably get rid of it by removing the --include-dependencies option when you import the app (that brings in some node-modules that aren't needed for the standard functionality)

pankova commented 3 years ago

@am-MongoDB is this done? I'd love to take this up 👍

Hello. I'm in progress 🙂

pankova commented 3 years ago

@am-MongoDB Sorry, could you check my steps? I have profilePic in the User database. After login, I see the avatar on screen with map and profile screen without navigating to other views.

I made a record: Screen Recording 2020-10-07 at 23.12.44.mov.zip

am-MongoDB commented 3 years ago

Hi @pankova – based on that video, I'm assuming that this isn't the first time that you've logged into this app in this simulator?

Try removing the app (which will clear any local Realm data) and then reinstall it. Once you log in, Realm starts syncing data, including the profile picture. That can take a number of seconds if there are a lot of existing reports to sync. The view isn't being automatically refereshed once the profile pic is available – it appears if (for example) you open the search view and then return to this one.

If you don't see this behaviour, then it might be because you don't have a large number of reports to sync?

Sheeri commented 3 years ago

Unassiging at @pankova 's request.

Dmitriy-Holovnia commented 2 years ago

It's realm side bug. On the first request, we don't get an image. It may have been fixed in the release version. JFY we are using 10.0.0-beta.3 for now, current release version is 10.18.0

Sheeri commented 2 years ago

OK. Let's talk tomorrow (Thursday) in the meeting about how to upgrade Realm for all three apps (ios, mobile, android).

Sheeri commented 2 years ago

According to https://github.com/WildAid/o-fish-ios/pull/454 we are at 10.0.19, so can we see if this is still a bug?