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

[BUG] List filtering not working after rotating device #4441

Closed
jesmrec opened this issue Jul 18, 2024 · 9 comments · Fixed by #4467
Closed

[BUG] List filtering not working after rotating device #4441

jesmrec opened this issue Jul 18, 2024 · 9 comments · Fixed by #4467

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Jul 18, 2024

Steps to reproduce

  1. Open list of files in portrait orientation
  2. Rotate device to landscape
  3. Click on lens icon
  4. Type something to filter the list

Actual behaviour

Filter does not work. When rotating the device again, filter does not work either. Required to kill the app an reopening to make it come to life again.

Expected behaviour

Filter works after (and before) rotating device

Pixel 2, Android 11
v4.3.0

@AwaisKhan128
Copy link
Contributor

AwaisKhan128 commented Aug 18, 2024

Is there someone already working on this bug? or may I start contributing into it? @jesmrec

@jesmrec
Copy link
Collaborator Author

jesmrec commented Aug 19, 2024

@AwaisKhan128 go ahead!!

Take a look to our CONTRIBUTING file, where you'll find guidelines about our development process (branch names, commit names...), so that your contribution will be aligned with the product itself.

Let us know any question or help you need!

@jesmrec
Copy link
Collaborator Author

jesmrec commented Aug 23, 2024

@AwaisKhan128 did you take a look? something we can help you?

@AwaisKhan128
Copy link
Contributor

Actually, I did try. First of all, gradle build failed due to its not recognizing the kotlin syntax in the gradle script.

Well, definitely, I will start over again on the weekend because, due to full-time commitment, I am unable to spare some time. @jesmrec

@jesmrec
Copy link
Collaborator Author

jesmrec commented Aug 23, 2024

Actually, I did try. First of all, gradle build failed due to its not recognizing the kotlin syntax in the gradle script.
Well, definitely, I will start over again on the weekend because, due to full-time commitment, I am unable to spare some time. @jesmrec

No worries, take your time! just for you to know that we are happy to help.

@AwaisKhan128
Copy link
Contributor

@jesmrec Could you tell me which server url used to continue in app?

@jesmrec
Copy link
Collaborator Author

jesmrec commented Aug 26, 2024

@jesmrec Could you tell me which server url used to continue in app?

You can use the production one: https://ocis.ocis.production.owncloud.works/

or the rolling one (latest master): https://ocis.ocis.master.owncloud.works/

with credentials: katherine/gemini

@AwaisKhan128
Copy link
Contributor

AwaisKhan128 commented Aug 26, 2024

@jesmrec Could you tell me which server url used to continue in app?

You can use the production one: https://ocis.ocis.production.owncloud.works/

or the rolling one (latest master): https://ocis.ocis.master.owncloud.works/

with credentials: katherine/gemini

Thanks. I take a look on it

@AwaisKhan128
Copy link
Contributor

Issue fixed. Soon creating a PR.

@jesmrec jesmrec linked a pull request Sep 2, 2024 that will close this issue
2 tasks
@JuancaG05 JuancaG05 self-assigned this Sep 3, 2024
@JuancaG05 JuancaG05 assigned JuancaG05 and unassigned JuancaG05 Sep 4, 2024
@JuancaG05 JuancaG05 linked a pull request Sep 6, 2024 that will close this issue
2 tasks
@JuancaG05 JuancaG05 linked a pull request Sep 18, 2024 that will close this issue
2 tasks
@jesmrec jesmrec removed the Sprint label Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants