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

Verify the OCS ID #85

Open
nickvergessen opened this issue May 7, 2015 · 5 comments
Open

Verify the OCS ID #85

nickvergessen opened this issue May 7, 2015 · 5 comments

Comments

@nickvergessen
Copy link
Contributor

The current entry of the ocsid is 166057 in all branches:
https://github.com/owncloud/search_lucene/blob/master/appinfo/info.xml#L44
https://github.com/owncloud/search_lucene/blob/stable8/appinfo/info.xml#L44
https://github.com/owncloud/search_lucene/blob/stable7/appinfo/info.xml#L34

However that app is only for oC 7: https://apps.owncloud.com/content/show.php?content=166057
oC 8.0 (and later**?**) should use 168709: https://apps.owncloud.com/content/show.php?content=168709

This causes problems, when people try to enable the app from the recommended section rather then the disabled section: owncloud/core#16092

Manually changing the ocsid resolves the issue.

@MorrisJobke
Copy link
Contributor

Manually changing the ocsid resolves the issue.

So we simply need to change this in stable8 and master to 168709?

@nickvergessen
Copy link
Contributor Author

well, master might need a new ocsid, im not familar with that.

@DeepDiver1975
Copy link
Contributor

well - one app - one id????

The app should be listed in the appstore only once as well?

hmmm - I don't get the concept - sorry

@nickvergessen
Copy link
Contributor Author

Well as far as I understand it, the app store can only contain one download for one ocsid. So we can not provide different versions depending on the requested oC version. Thats why "they" started to use multiple entries for each version.

@ghost
Copy link

ghost commented Jan 2, 2016

Mhhh, that should be now fixed for quite some time. The appstore can now provide different downloads / versions for one single ocsid so it should be safe to use the new 168709 ocsid.

Another issue is, that the "All versions" download at https://apps.owncloud.com/content/show.php?content=168709 is still providing the old 166057 ocsid. Have raised #113 here separately.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants