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

v2.4.10-beta1 report #1280

Closed
kri164 opened this issue Aug 28, 2022 · 5 comments
Closed

v2.4.10-beta1 report #1280

kri164 opened this issue Aug 28, 2022 · 5 comments
Labels

Comments

@kri164
Copy link

kri164 commented Aug 28, 2022

Hello Andrew,

there are few issues with last beta version:

  • string Enable Map rotation is not translated. I cannt find it on POEditor.com
  • the second page of Welcome wizard is not scrollable and doesnt display all text
  • when opening the section Settings -> Map, it has wrong headline description (Notification instead Map)
  • when open Friends and click to some name, it doesnt show friend position on map. It was working in previous versions.
  • when export configuration, it reports Export failed eveytime not depend on result of export.

Regards

@growse
Copy link
Collaborator

growse commented Sep 3, 2022

Hi!

Thanks for letting me know. Hopefully these are just minor regressions, but I'll take a look at each:

string Enable Map rotation is not translated. I cannt find it on POEditor.com

Ah, yes, I forgot to sync back up to POEditor after adding some resources. Have now synced. I see some translations have already been updated, so folded them in on d373b8e

the second page of Welcome wizard is not scrollable and doesnt display all text

Youch, I should test on smaller screen sizes :) Should now be fixed in 29f7c43

when opening the section Settings -> Map, it has wrong headline description (Notification instead Map)

Good spot, fixed in 413b6a2

when open Friends and click to some name, it doesnt show friend position on map. It was working in previous versions.

I saw this too, and I think I fixed this already in 0b739fc

when export configuration, it reports Export failed eveytime not depend on result of export.

This has been flaky for a while, it needs some more in-depth love.

I'll cut another beta now. Thanks!

@growse growse added the bug label Sep 3, 2022
@kri164
Copy link
Author

kri164 commented Sep 3, 2022 via email

@growse
Copy link
Collaborator

growse commented Sep 13, 2022

when export configuration, it reports Export failed eveytime not depend on result of export.

Just an update on this. The problem here is that OT only gets to see a simple "Activity Result Code" from the sharing. When you hit "export", and it pops up a dialog to share with, clicking on an app starts a new Activity. It's sort of up to that application to correctly report the status of being given some data that's been shared with it.

From testing, for apps which behave "properly" (e.g. Google Drive), any action that results in the file not being saved to Google Drive returns a "Cancelled" message back to OT, which then displays that "failed" message.

Which app are you sharing with? I'll see if I can test with that one explicitly to work out what's going on.

@kri164
Copy link
Author

kri164 commented Oct 11, 2022 via email

@growse
Copy link
Collaborator

growse commented Jan 5, 2023

This has been flaky for a while, it needs some more in-depth love.

I've dropped the error message, as the "canceled" result is not necessarily an error.

I think with that and #1379, that covers everything here?

growse added a commit that referenced this issue Jan 5, 2023
…isn't actually reached

Resolveds last point on #1280
@growse growse closed this as completed Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants