arnovr/ocusagecharts

Use integer version numbers

Closed this issue · 4 comments

Hi,

currently three different versions are used by the app:

3 in https://github.com/arnovr/ocusagecharts/blob/master/appinfo/version
1.0 in https://github.com/arnovr/ocusagecharts/blob/master/appinfo/info.xml#L11
0.3.1 in http://apps.owncloud.com/content/show.php/Storage+Usage+%2B+Activity+Charts?content=166746 and https://github.com/arnovr/ocusagecharts/releases

I don't know if this can cause problems but i think the version numbering should be integer between all files.

As a follow-up:

This caused messages when trying to install the app from within ownCloud:

{"reqId":"5436c3eb210dc","app":"core","message":"App can't be installed because the version in info.xml/version is not the same as the version reported from the app store","level":3,"time":"2014-10-09T17:20:43+00:00","method":"POST","url":"\/index.php\/settings\/ajax\/enableapp.php"}
  • version is kept on 3 ( should only be used on database updates according to documentation )
  • Forgot to update info.xml ( oops! ) to make consistent, made it to 3 aswell
  • Updated apps.owncloud.com to also visually have version 3.
  • kept github like it is.

Let me know if it is fixed!

Hi,

and thanks for the update. My app page is only showing 3rd party apps from time to time so i can't test this at the moment but let you know later.

Hi,

installation directly out of the app list of OC is working now as expected. Closing this as fixed. 👍