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

project is missing after trying to publish then... #1470

Closed
door43app opened this issue May 11, 2016 · 5 comments
Closed

project is missing after trying to publish then... #1470

door43app opened this issue May 11, 2016 · 5 comments

Comments

@door43app
Copy link

Notes

project is missing after trying to publish then app crashes when rotating while merge conflict dialog is open

Environment

Environment Key Value
version 7.0
build 128
UDID 29fc019efc78c5c1
Android Release 5.1.1
Android SDK 22
Brand OnePlus
Device OnePlus

Log history

5/11/16 9:03 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_2jn_text_reg
5/11/16 9:03 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_1jn_text_reg
5/11/16 9:01 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_2jn_text_reg
5/11/16 9:01 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_1jn_text_reg
5/11/16 9:00 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_1jn_text_reg
5/11/16 8:57 AM  W/com.door43.translationstudio.tasks.UpdateAppTask: Failed to migrate the target translation aa_1jn_text_reg
@da1nerd
Copy link
Contributor

da1nerd commented May 11, 2016

this is because the manifest had a merge conflict in it which corrupted the target translation.

@da1nerd
Copy link
Contributor

da1nerd commented May 11, 2016

as a precaution the apps should always make sure the master branch when performing migrations. And the merging should be done in a temporary branch.

@da1nerd
Copy link
Contributor

da1nerd commented May 11, 2016

for simplicity this was resolved by just choosing --ours for the manfiest.json when it has a merge conflict.

@cckozie
Copy link

cckozie commented May 25, 2016

@neutrinog Is there something to be tested on this one?

@da1nerd
Copy link
Contributor

da1nerd commented May 25, 2016

I thought this was already tested. You'll need to create a merge conflict between the server and the app. You can just use what you've been doing to test the merging when importing except you'll be uploading them to the server.

A merge conflict dialog should appear when when you try to upload the last one. Rotate the device at this point and it should not crash.

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

No branches or pull requests

3 participants