Sounds good but I would remove the code on the master branch and just leave the README, adding a little explanation that it’s only for maintaining old branches and that the master was moved to platform. WDYT? Thanks -Vincent
On 11 Jul 2016, at 15:51, Anca Luca <[email protected]> wrote:
Hello,
here is what was done on the ratings application for this release: * a branch was created on contrib for maintaining the bugfix old version: https://github.com/xwiki-contrib/application-ratings/tree/stable-1.3.x * version 1.3.5 was released from this branch * for the 3 problems fixed on this version, the following was done: - one issue did not reproduce on the latest version of platform ratings - the one from commit 07758a698d400ed275cde1e6fde4515d41711b40 : the fact that saving ratings in different spaces did not work - a second issue did reproduce and was reported (the one fixed in commit cd2c190e22f709e2b55fed19be25ea8ef7631730), but was not yet fixed: http://jira.xwiki.org/browse/XWIKI-13545 . However, this is not a critical bug (not even sure if it's a bug, is more a configuration that is not followed). It would need to be analysed and defined the correct desired behaviour in the ratings application - the third issue reproduced, was reported and fixed in http://jira.xwiki.org/browse/XWIKI-13543 (the one from commit 4a5ed2ca13d1a01962a7d9edeb8b34134a6ad704 ).
Hope this is acceptable for the contrib rules.
Note: all the work was done by https://github.com/rstavro , I am just the messenger.
Thanks, Anca
On Wed, Jun 29, 2016 at 11:51 AM, Ecaterina Moraru (Valica) < [email protected]> wrote:
I agree to release this version, only if it's tested and fixed on Platform too (8.x+ or even 7.4.x+). Only in this case we could consider it to be a bugfix release. And if we don't do the testing now, it will not be done at all, thus forking the code.
To be honest, I would prefer to delete / remove the commits rights for applications that we move in Platform or not move things in Platform anymore and keep separate version for them (... etc the whole discussion about Contrib).
Thanks, Caty
On Wed, Jun 29, 2016 at 12:45 PM, Anca Luca <[email protected]> wrote:
Hello all,
while using the ratings application on an XWiki 6.2.4, we encountered some bugs related to the storage of ratings in a separate space and with the update of a user's rating.
We fixed them in these 2 commits:
https://github.com/xwiki-contrib/application-ratings/commit/07758a698d400ed2...
and
https://github.com/xwiki-contrib/application-ratings/commit/cd2c190e22f709e2...
.
Now, the contrib ratings application was indeed "retired" so we normally should not use the contrib repo anymore. However, the work in these 2 commits is about fixing bugs (which might even be affecting the most recent version of ratings - we need to check that, didn't check it yet), and I don't see exactly what other option we have in this case (note that we cannot switch to the "platform ratings" as we're on XWiki 6.2.4 and ratings are in platform since 6.4.
So, this is what I propose: * release a 1.3.5 version from the contrib code - if I understand correctly how extensions.xwiki.org works, this release will not appear as "most recent" on the application page - if I understand correctly how maven works, this version will not be considered more recent than 8.1 even if released after, because of the version number. * this means that the version 1.3.5 will be available as an upgrade to the users that use the contrib extension (v 1.3.4 or lower), will be available for new install to all users for which the dependencies requirements are satisfied and will not impact the users which use the platform version of the ratings app. Please let me know if I'm wrong in my reasoning.
The advantage of this release would be publishing the bugfix work that we've done and making it available to all users of the ratings extension on versions lower than 6.4. Also, for us the advantage would be not having to deal with a code fork and manage a release / snapshot of this forked code separately.
Unless somebody -1s this in the very close feature, we will go ahead with the release.
Thanks, Anca
P.S. There is a larger discussion to have about how to provide bugfixes on older versions of platform applications without upgrading all the wiki, but this should be in another mail or on IRC (e.g. if we discover that indeed these issues affect ratings app, the platform version, and we make fixes for these bugs, how can we provide these fixes to an XWiki 6.4 user?) _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs