+1

Thanks,
Caty

On Thu, Jun 6, 2019 at 4:35 PM Vincent Massol <[email protected]> wrote:

> So far:4 +1, no 0 and no -1.
>
> Because we need to release 11.3.1 ASAP i’ve applied Stephane’s PR and
> merged it on 11.3.x branch.
>
> Thanks
> -Vincent
>
> > On 6 Jun 2019, at 14:32, Thomas Mortagne <[email protected]>
> wrote:
> >
> > +1, it's new enough and the current one is really not great
> >
> > Should be cherry-picked in 11.3.x branch to make the change effective
> ASAP.
> >
> > On Thu, Jun 6, 2019 at 12:54 PM Stéphane Laurière <[email protected]>
> wrote:
> >>
> >> Hi devs,
> >>
> >> I'm opening this vote in order to propose an API breakage consisting in
> renaming the extension point "org.xwiki.platform.head" to
> "org.xwiki.platform.html.head". The former identifier was introduced too
> early by mistake in 11.1RC1 due to an omission by me, apologies, but it
> does not fit well because it does not self-explain that it targets the html
> head. Here is the extension point documentation page:
> >>
> >>
> https://www.xwiki.org/xwiki/bin/view/Documentation/DevGuide/ExtensionPoint/HTMLHead
> >>
> >> As far as I know, there's just one public extension using this
> extension point for now, it's the OpenGraph Application, which already uses
> the new identifier:
> >>
> >>
> https://extensions.xwiki.org/xwiki/bin/view/Extension/OpenGraph%20Application/
> >>
> >> Further information about the change is available in the following pull
> request, which requires among other things that this vote passes:
> >>
> >>   https://github.com/xwiki/xwiki-platform/pull/1115
> >>
> >> Thanks to Caty for spotting the issue and the guidance on how to fix
> it, and to Simon and Vincent.
> >>
> >> Kind regards
> >>
> >> Stéphane
> >>
> >>
> >> --
> >> Stéphane Laurière
> >> XWiki – https://xwiki.com
> >>
> >
> >
> > --
> > Thomas Mortagne
>
>

Reply via email to