Pierre Goiffon

Results 132 comments of Pierre Goiffon

Yes, I did reproduce the issue : 1. create a new lmhosts entry 2. run setup using an URL with this entry 3. remove the entry (comment lmhosts declaration +...

Pushed 2 branches with equivalent commits : * issue/3677_url_31 based on support/3.1 * issue/3677_url30 base on support/30 Seems like solely changing app_root_url (not running the setup again) is OK now...

Note that : * display template were removed in 3.0.0 with N°3233 * the icon compilation is done now in MFCompiler::GenerateStyleDataFromNode, which calls an ormStyle

Hello, Indeed we asked to rebase the branch on support/3.0. Currently the branch start is 42599eae which is in develop... Can you do the rebase Thomas ?

Hello Jeffrey, reviews occurs every month, first week the technical one, second week the functional one.

Functional review : Only one zlist for all transitions ? Simpler, yes do it ! But don't forget the fallback for forgotten fields (not defined in XML but present in...

Accepted during Functional Review. Warning : the current screenshot of a transition with the fix isn't up to date ! Anne Catherine did some changes to handle Guillaume feedbacks, they...

> Off this PR, a discussion should take place with Eric and Pierre to discuss about where the triggers should be (within the listener?) and how to handle their errors....

Hello, 2 year have passed, all of our apologies :/ But for this PR to be processed, we need a way to reproduce easily, and now it seems to me...