Skip to content
This repository has been archived by the owner on Feb 13, 2024. It is now read-only.

feat: add wopi validator helper command #13

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DeepDiver1975
Copy link
Collaborator

@DeepDiver1975 DeepDiver1975 commented Nov 28, 2023

This command does:

  • upload test.wopitest to the give user
  • get WOPISrc url and access token via OpenInApp API
  • launches the wopi-validator docker image

For ci integration we most probably cannot run the docker image - will be taken care of once we are there ....

Supporting testgroup argument

MICRO_REGISTRY=mdns ./wopi-validator -u admin -p admin -g BaseWopiViewing

@DeepDiver1975 DeepDiver1975 marked this pull request as draft November 28, 2023 11:19
if err != nil {
return err
}
return nil
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the file in place, you now can call app.GatewayAPIClient.OpenInApp(...)
It's the cs3api equivalent of the /app/open call in the bash setup.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Already doing this.....

@DeepDiver1975 DeepDiver1975 force-pushed the feat/wopi-validator-helper branch 3 times, most recently from 6287eef to 24fd059 Compare November 28, 2023 15:17
@DeepDiver1975 DeepDiver1975 marked this pull request as ready for review November 28, 2023 15:17
@DeepDiver1975
Copy link
Collaborator Author

adding command arguments -nand -g ....

@wkloucek wkloucek removed their request for review February 13, 2024 08:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants