On Wed, Nov 16, 2016 at 7:54 AM, Sean Carte <[email protected]> wrote:

>
> There's already been a successful test and merging is (now) blocked, so I
> wasn't able to merge the PR, but I did make the change manually (after
> removing Wandi's fix), and it does appear to work.
>

Please, comment on the PR that you tested it. Others who will look at the
PR are unlikely to look here for that information.


As far as I can tell, the documentation also needs an update:
>
> https://wiki.duraspace.org/display/DSDOC6x/ORCID+
> Integration#ORCIDIntegration-Configuration
>
> The final part of configuration is to add the authority consumer in front
> of the list of event consumers. Add "authority" in front of the list as
> displayed below.
> event.dispatcher.default.consumers = authority, versioning, discovery,
> eperson, harvester
>
> I find I have to remove 'harvester' from the line above:
>
> event.dispatcher.default.consumers = authority, versioning, discovery,
> eperson
>

Why did you have to remove harvester? Shouldn't be necessary, but I haven't
tried.


On Wed, Nov 16, 2016 at 9:04 AM, Wandi Tobosi <[email protected]> wrote:

> True my fix seems to also have effects on the handlebar templates which
> renders
> the advanced search filters
>

If there are any negative effects, can you please comment on the PR what
they are? Thanks.


Regards,
~~helix84

Compulsory reading: DSpace Mailing List Etiquette
https://wiki.duraspace.org/display/DSPACE/Mailing+List+Etiquette

-- 
You received this message because you are subscribed to the Google Groups 
"DSpace Technical Support" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/dspace-tech.
For more options, visit https://groups.google.com/d/optout.

Reply via email to