Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Add summary, category, doc, website, bugs, repository; change descrip…#62

Merged
pmaier1 merged 2 commits intorelease/0.2.0from
pmaier1-patch-1
Feb 12, 2018
Merged

Add summary, category, doc, website, bugs, repository; change descrip…#62
pmaier1 merged 2 commits intorelease/0.2.0from
pmaier1-patch-1

Conversation

@pmaier1
Copy link

@pmaier1 pmaier1 commented Jan 16, 2018

…tion

Screenshot/cover image still missing. @hurradieweltgehtunter

@pmaier1 pmaier1 added this to the QA milestone Jan 16, 2018
@pmaier1 pmaier1 self-assigned this Jan 16, 2018
@pmaier1 pmaier1 requested review from PVince81 and VicDeo January 16, 2018 10:15
@patrickjahns patrickjahns mentioned this pull request Jan 16, 2018
19 tasks
Copy link
Member

@VicDeo VicDeo left a comment

Choose a reason for hiding this comment

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

Description looks good to me.

Screenshot is a required appinfo entry, marketplace won't accept the package with no screenshot.

@patrickjahns
Copy link
Contributor

@pmaier1
keep this open and include screenshot, or shall we have a separate PR for screenshot

@hurradieweltgehtunter
Copy link

hurradieweltgehtunter commented Feb 6, 2018

Sorry guys, took me too long. Any opinions on that?
Compare to https://marketplace.owncloud.com/apps/user_ldap
I'm trying to set a standardized style for free apps like the non-free apps have already.
owncloud-app-email-template-editor

@hurradieweltgehtunter
Copy link

Update
owncloud-app-email-template-editor

@pmaier1
Copy link
Author

pmaier1 commented Feb 12, 2018

Added the second screenshot. Ready to merge from my side.

@pmaier1 pmaier1 merged commit cf3e896 into release/0.2.0 Feb 12, 2018
@VicDeo VicDeo deleted the pmaier1-patch-1 branch February 12, 2018 20:06
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.

4 participants