syncope-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Colm O hEigeartaigh <cohei...@apache.org>
Subject Re: Some comments on Connectors
Date Wed, 23 Mar 2016 16:55:35 GMT
On Wed, Mar 23, 2016 at 4:35 PM, Francesco Chicchiriccò <ilgrosso@apache.org
> wrote:

>
> a) I filed a bug (SYNCOPE-805) for when the user doesn't specify a
>> destination realm for a pull task. However, I wonder if it would it be
>> possible for the user to select the destination realm from a drop-down
>> list?
>>
>
> Sounds good: can you suggest this as alternative fix for SYNCOPE-805?
>

Done.


>
> b) When running Syncope via a Maven archetype + mvn -Pembedded (using the
>> core main, not test, configuration) I don't see anything relating to
>> "Extensions" in the UI. I see it when I download the standalone
>> distribution though. Perhaps the default configuration for the archetype
>> needs to be updated to reference Camel etc?
>>
>
> The idea is to not enable extensions (nor Camel nor Swagger UI, which are
> the extensions currently available) by default; maybe an idea is to add a
> further "all" profile - similarly to what we do with fit/core-reference so
> that one can run
>
> * mvn -Pembedded to get the environment without extensions
> * mvn -Pembedded,all to get the environment with extensions
>

Yep that'd be great.


>
> WDYT?
>
> c) I noticed here (
>> https://dist.apache.org/repos/dist/dev/syncope/2.0.0-M2/)
>> that half of the artifacts start with "apache-syncope" and the other half
>> with "syncope". Any reason why they are not all starting with
>> "apache-syncope" for consistency?
>>
>
> Essentially, .deb packages have the "apache-syncope" prefix rather than
> "syncope" - as all other artifacts, which take the name off of Maven
> modules, do instead.
>
> We might want to change the .deb packages prefix to "syncope" (and
> generated directories as /etc/apache-syncope, /var/log/apache-syncope etc)
> for consistency; anyway, this is following the practice we have since 1.2.0.
>

Ok thanks for the explanation. TBH I wouldn't bother changing things, it's
just a trivial issue really.

Colm.


>
> Regards.
>
>
> On Wed, Mar 23, 2016 at 3:56 PM, Colm O hEigeartaigh <coheigea@apache.org>
>> wrote:
>>
>> Answers inline:
>>>
>>> On Tue, Mar 22, 2016 at 3:49 PM, Francesco Chicchiriccò <
>>> ilgrosso@apache.org> wrote:
>>>
>>> On 22/03/2016 15:09, Colm O hEigeartaigh wrote:
>>>>
>>>> a) In the "Capabilities" tab, the header is "capabilitiesPalette" which
>>>>> seems a bit unnecessary IMO. An "all" checkbox would possibly be useful
>>>>> here which would select all of the other checkboxes.
>>>>>
>>>>> Agree: would you mind opening an improvement for this?
>>>>
>>> https://issues.apache.org/jira/browse/SYNCOPE-802
>>>
>>>
>>> b) In the "Configuration" tab, it's not clear to me what the On/Off
>>>>> buttons
>>>>> mean?
>>>>>
>>>>
>>>> As for 1.2, such buttons sets the possibility to override the related
>>>> properties in the resources using the given connector.
>>>> I agree it is not clear now: again, could you please take care of filing
>>>> another improvement for this?
>>>>
>>> https://issues.apache.org/jira/browse/SYNCOPE-803
>>>
>>> c) In previous Syncope versions, one could hover over the textfields IIRC
>>>>
>>>>> and get an explanation as to what the configuration item meant. However
>>>>> this is not working, unless I am missing something.
>>>>>
>>>>> Once more, I agree with you; third required improvement in a row.
>>>>
>>>> https://issues.apache.org/jira/browse/SYNCOPE-804
>>>
>>> d) Not related to Connectors, but I noticed that all of the table
>>>> listings
>>>>
>>>>> in the UI have both the header and footer containing the column names.
>>>>> IMO
>>>>> it would be sufficient just to have the header there.
>>>>>
>>>>> This was set according to the Admin LTE template that we have chosen
>>>> for
>>>> the new admin console: see [1]; however, as you can imagine, we can
>>>> easily
>>>> revert to the single-header version; it's more a matter of taste.
>>>>
>>> Sure, it's not an important issue anyway.
>>>
>>> Colm.
>>>
>>> WDYT?
>>>>
>>>> Regards.
>>>>
>>>> [1] https://almsaeedstudio.com/themes/AdminLTE/pages/tables/data.html
>>>>
>>>
> --
> Francesco Chicchiriccò
>
> Tirasa - Open Source Excellence
> http://www.tirasa.net/
>
> Involved at The Apache Software Foundation:
> member, Syncope PMC chair, Cocoon PMC, Olingo PMC, CXF committer
> http://home.apache.org/~ilgrosso/
>
>
>


-- 
Colm O hEigeartaigh

Talend Community Coder
http://coders.talend.com

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message