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.

Sign up for GitHub

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

Jump to bottom

[BUG] In pdf preview, Go to Page don't take user to last page #1033

Closed
itsaboutcode opened this issue Sep 10, 2021 · 1 comment · Fixed by #1034
Closed

[BUG] In pdf preview, Go to Page don't take user to last page #1033

itsaboutcode opened this issue Sep 10, 2021 · 1 comment · Fixed by #1034
Milestone
11.7.1

Comments

@itsaboutcode
Copy link

Steps to reproduce

  1. Open PDF
  2. Select "Go to page" action
  3. Enter last page# and it will show you an error saying "The entered page number doesn't exist"

Expected behavior

It should take the user to the last page of the PDF.

Actual behavior

It's giving an error "The entered page number doesn't exist"

Client

iOS version: 14.5

ownCloud app version: 11.7.0

Device model: Simulator

Server configuration

Web server: NA

Database: NA

PHP version: NA

ownCloud version: NA

Storage backend (external storage): NA

Logs

Web server error log

NA

ownCloud log (data/owncloud.log)

NA

Tell us what could be improved:
itsaboutcode added a commit to itsaboutcode/ios-app that referenced this issue Sep 10, 2021
@itsaboutcode
owncloud#1033 - In pdf preview, Go to Page don't take user to last page
dbcf4f5
@itsaboutcode itsaboutcode mentioned this issue Sep 10, 2021
Merged
10 tasks
@hosy hosy linked a pull request Sep 15, 2021 that will close this issue
#1033 - In pdf preview, Go to Page don't take user to last page #1034
Merged
10 tasks
@hosy hosy added this to the 11.7.1-Fix milestone Sep 15, 2021
hosy added a commit that referenced this issue Sep 16, 2021
@itsaboutcode @mneuwert @hosy
#1033 - In pdf preview, Go to Page don't take user to last page (#1034)
ea67878 
* Added associated domains to resign script (#1028)

* First version of the associated domains addition feature in the resign script

* - lines from domains.txt were not correct read
- highlighting output
- fixed wrong provisioning path
- changed version number
- updated readme file

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* #1033 - In pdf preview, Go to Page don't take user to last page

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>
@jesmrec
Copy link
Contributor

jesmrec commented Sep 17, 2021

Done. Ready to 11.7.1

@jesmrec jesmrec closed this as completed Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
11.7.1
Development

Successfully merging a pull request may close this issue.

#1033 - In pdf preview, Go to Page don't take user to last page itsaboutcode/ios-app
3 participants
@itsaboutcode @hosy @jesmrec