Skip to content
This repository was archived by the owner on Oct 31, 2018. It is now read-only.

Conversation

@karakayasemi
Copy link
Contributor

-The screenshot has been updated with the suggestion of @kawohl
-Bug tracker and website tags are added for better market support

<prelogin/>
</types>
<category>security</category>
<screenshot>https://raw.githubusercontent.com/owncloud/security/43c55325cf09e5a8b81fafc358b08701aba81173/screenshots/settings.png</screenshot>
Copy link
Contributor

Choose a reason for hiding this comment

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

there can be multiple screenshots - keep it

Copy link
Contributor

Choose a reason for hiding this comment

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

but as second entry - the first one is used as the main image on the marketplace

@codecov-io
Copy link

codecov-io commented Nov 22, 2017

Codecov Report

Merging #25 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage     70.56%   70.56%           
  Complexity       70       70           
=========================================
  Files            12       12           
  Lines           282      282           
=========================================
  Hits            199      199           
  Misses           83       83

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b614117...94ec259. Read the comment docs.

@karakayasemi
Copy link
Contributor Author

@DeepDiver1975 is it ready to merge?

@DeepDiver1975 DeepDiver1975 merged commit b8be105 into master Dec 15, 2017
@DeepDiver1975 DeepDiver1975 deleted the better-market-support branch December 15, 2017 10:13
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