After I uploaded the package to the oc marketplace, I got a mail that it violates the marketplace guidelines (reference to a cloud storage provider which is not oc... :roll_eyes: ).
@dartcafe Am I right that you have a oc instance running? If so, could you add one screenshot for oc?
v1r0x
All 11 comments
I can make some screenshots, or we just cut out the NC header.
dartcafe
on 25 Oct 2017
We should also add all the screenshots to the info.xml file.
splitt3r
on 25 Oct 2017
222. I just removed the header.
dartcafe
on 25 Oct 2017
Hmm...I'm not happy with removing the NC header just because the OC marketplace has annoying restrictions. I'll check if it is possible to select another screenshot in the marketplace or rather drop OC support than live with their restrictions (I mean restrictions were all other NC users have restrictions too, such as not that cool screenshots). But that's just my opinion. What do you think?
v1r0x
on 26 Oct 2017
To have some good screenshots, I have to build up representative polls. Doing it in two instances, is annoing. So removing the header is a acceptable agreement for me, besides the manual work of the CSS files.
But as @splitt3r mentioned here #178, reworking the mail will cause more doubling.
On the other hand, the release is usable in OC as well, even if it is not in the OC app store.
dartcafe
on 26 Oct 2017
We could create screenshots with some -nc / -oc naming convention and replace the urls in the info.xml file in our Makefile.
One thing i don't like about the screenshots without header is that the theme colours don't match the Nc / oC main colours and so they look out of place in one of the appstores.
splitt3r
on 26 Oct 2017
I will think about some database imports for showcases.
dartcafe
on 27 Oct 2017
We could create screenshots with some -nc / -oc naming convention and replace the urls in the info.xml file in our Makefile.
I think that's a good compromise.
v1r0x
on 27 Oct 2017
@v1r0x @dartcafe https://github.com/nextcloud/polls/pull/224/files gave it a try. What do you think?
splitt3r
on 28 Oct 2017
They seem to use the Nc + oC logo + some theming trick. Only people who really care will notice the differences so i like that solution a lot.
splitt3r
on 15 Nov 2017
Solved by #257 ref #259
splitt3r
on 3 Dec 2017
Related issues
jacotec
路
3Comments
brtptrs
路
4Comments
amittel
路
5Comments
wiertz
路
5Comments
wompydomp
路
6Comments
Most helpful comment
Hmm...I'm not happy with removing the NC header just because the OC marketplace has annoying restrictions. I'll check if it is possible to select another screenshot in the marketplace or rather drop OC support than live with their restrictions (I mean restrictions were all other NC users have restrictions too, such as not that cool screenshots). But that's just my opinion. What do you think?