Hi Greg, 

> Further to Chris' comments, I'm not
> sure I entirely understand what the
> point of these changes would be.  Is this to be able to
> find workflows
> faster? 
No. I assume this list will have no more than 100 entries (I guess there will 
be up to 20). No need to make it faster.

> To be able to filter them when the UI tries to
> show a list to
> the user?


It's only a filter. Simple filter, java code based filter, the only changes I 
am expecting are:

1.
WorkflowServiceImpl.java :: method: listAvailableWorkflowDefinitions.

2.
Optional parameter to the endpoint.

3.
Workflows parser & storage - a support for node: <tags>  with data type of 
NMTOKEN (For simplicity).
All optional, so it will not have affect on installation that have their own 
workflow definitions or their own JQuery code to access the list of workflows 
without this optional parameter.



> I don't have a problem with this as long as you can
> provision your own
> development resources. 

Hey! If this would be the case, I would just fill in a bug report! :-)
Or comment on 8780  :-)
Of course, I will attach a patch to MH-8780.
Just need to make sure nobody is working on similar feature.


BTW:
As mentioned: We are starting slow, because we want to contribute features that 
will improve the server (I hope this one will). 
I have started with typos etc. Now we are moving forward with small things like 
this one.



> I'm just not sure I understand
> what everyone
> gets out of this if you create this patch :)

First:
(1) it solves 8780
(2) allows filtering workflow definitions.
   different set of workflows for scheduler
   different set of workflows for upload page
   different set of workflows for capture agent 

------ and then way ahead: ----
Assume Matterhorn will provide an interface (episode service) for users that 
want to operate on published videos. 

 (3) it can be easily extended to handle filtering by role...
     all that needs to be added it support for user role in 
 WorkflowServiceImpl.java :: method: listAvailableWorkflowDefinitions.
     + some clear rules how to add appropriate tags, and there you go. 



I will try to post details later this week.


_______________________________________________
Matterhorn mailing list
[email protected]
http://lists.opencastproject.org/mailman/listinfo/matterhorn


To unsubscribe please email
[email protected]
_______________________________________________

Reply via email to