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

Add support for non-dav files #1338

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Add support for non-dav files #1338

merged 1 commit into from
Aug 23, 2022

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Aug 18, 2022

Called like

  const fileInfo = {
	filename: '/path/to/file.jpg',
	basename: 'file.jpg',
	mime: 'image/jpeg',
	etag: 'xyz987',
	hasPreview: true,
	fileid: 13579,
  }
  OCA.Viewer.open({
	fileinfo: fileInfo,
	list: [fileInfo],
  })
  • Adjust open so it can work with a path or a given fileInfo -> fetch file info only for paths
  • Get rid of url file info property and supply the non-dav URL as filename source
  • Specify required file info properties
  • Update readme documentation
  • Adjust specific viewer extensions, if necessary
  • Follow-up: get rid of unused yet required etag property

@skjnldsv
Copy link
Member

@matiasdelellis why the laughing emoji? 😁

src/views/Viewer.vue Outdated Show resolved Hide resolved
@skjnldsv skjnldsv force-pushed the enhancement/non-dav-files branch 3 times, most recently from 733cee7 to 011d2a0 Compare August 19, 2022 19:09
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 23, 2022
@ChristophWurst ChristophWurst marked this pull request as ready for review August 23, 2022 13:26
@skjnldsv
Copy link
Member

/rebase

@skjnldsv
Copy link
Member

/compile amend /

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 23, 2022
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
Development

Successfully merging this pull request may close these issues.

2 participants