-
Notifications
You must be signed in to change notification settings - Fork 76
Conversation
By analyzing the blame information on this pull request, we identified @DeepDiver1975, @ChristophWurst and @drfuture to be potential reviewers |
@tahaalibra please set the labels appropriately when submitting pull requests, thanks :) |
please add a changelog entry for this fix |
68e160f
to
50428f1
Compare
@viki53 you volunteered for testing, can you test with innodb please? |
needs rebase |
50428f1
to
b5727ff
Compare
@ChristophWurst done!! |
👍 thanks |
@ChristophWurst Sure! Which branch/commit should I clone to test? |
this one ;-) |
I just tried to install the app. DB upgrade went fine but then I got an OwnCloud error when I accessed the Web interface. Now that I fixed that by upgrading other apps (which made OwnCloud deactivate all the apps), I can't enable the Mail app because it "makes the server unstable" apparently… |
could be a duplicate https://github.com/owncloud/mail/issues/1522. |
I posted a comment on the #1522 issue with all the details I can get. Let me know if it helps |
All good! 👍 Upgrade worked fine, app seems to work as expected (at least for the retrieving mails part). Thanks for the fix! BTW, will the app update as usual even though I installed via a manual download? Or will I need to replace it with a version from the store? |
@viki53 the version in the store is signed and you should get the next updated when it's available. I don't think you need to replace it now. It should be fine :-) |
change email length from 64 to 255 in the database
@ChristophWurst @Gomez