11 Jul
2016
11 Jul
'16
3:55 p.m.
Looks good. Note that we could make master branch "protected", you can
forbid any push on protected branches (except for owners who can
always push).
On Mon, Jul 11, 2016 at 3:51 PM, Anca Luca <lucaa(a)xwiki.com> 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) <
valicac(a)gmail.com> wrote:
--
Thomas Mortagne
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 <lucaa(a)xwiki.com> wrote:
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs 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/07758a698d400ed…
and
https://github.com/xwiki-contrib/application-ratings/commit/cd2c190e22f709e…
.
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
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs