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

Reload rich document edition on reload event from webview (triggered on session expiration) #12814

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nicofrand
Copy link

This fixes the android part of #12138.
The webview part needs some tweaking but can already be used with this PR for testing: https://github.com/nicofrand/nextcloud-text/commits/fixReconnect/.

I am quite new to android development and brand new to this app (thanks @juliushaertl for all the help already). I don't know if tests should be added or how to put them. I am open to discussion ofc.

To test (see #12138):

  • Use this version of the Text app that will trigger a reload event when a 403 error is caught: https://github.com/nicofrand/nextcloud-text/commits/fixReconnect/
  • Open a markdown file in the android app
  • Put the app in background and wait a few minutes (so the session expires due to lack of sync requests sent while in background)
  • Go back to app: the app should reload
  • Tests written, or not not needed

@juliushaertl juliushaertl added bug 3rdparty An upstream, OS, dependency, or other third-party matter labels Apr 12, 2024
@juliushaertl
Copy link
Member

@nicofrand Maybe you can also already open a pull request for the text part for easier testing and early review :)

@nicofrand
Copy link
Author

@nicofrand Maybe you can also already open a pull request for the text part for easier testing and early review :)

Done: nextcloud/text#5662
There are still things to improve though on the Text app side.

Copy link

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

@nicofrand
Copy link
Author

It's a bit ironic to get a survey about a pull request that did not receive any feedback…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rdparty An upstream, OS, dependency, or other third-party matter 3. to review bug feature: editor feedback-requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants