-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add mdpresentation-viewer app to apps.json #11
Conversation
c1b6602
to
6d3fe2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be under webApps/jankaritech/mdpresentation-viewer/cover.png
as mentioned in #8 (review) ?
CC @kulmann
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should webApps/<org-name>/<repo-name>
be strict? if yes then the path would become webApps/jankaritech/web-app-presentation-viewer/cover.png
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved to webApps/jankaritech/web-app-presentation-viewer/
6d3fe2a
to
c3486cf
Compare
c3486cf
to
cfe0bbb
Compare
I didn't notice this PR, sorry 🙈 waiting for feedback about default CSP rules, see owncloud/ocis#9993 - if that's getting included in the next ocis rolling release (tomorrow) we can merge here. Otherwise I'd like to add some description about the necessity to add a CSP rule. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! 🥳
Added mdpresentation-viewer app.
Closes JankariTech/web-app-presentation-viewer#67