can i just use get the application object and call listTags on that? adding another level of indirection is confusing: are  the tags in a display different from those in the application? at what level does the maximum display take effect? right now the DisplayManager processes the max display, but shouldn't that really have happened on the server side?

From: Jonathan Johnson <notifications@github.com<mailto:notifications@github.com>>
Reply-To: Jazzee/Jazzee <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, September 19, 2013 1:32 PM
To: Jazzee/Jazzee <Jazzee@noreply.github.com<mailto:Jazzee@noreply.github.com>>
Cc: larry roberts <lawrence.roberts@ucsf.edu<mailto:lawrence.roberts@ucsf.edu>>
Subject: Re: [Jazzee] added tag choosing to the grid (#245)


In src/controllers/applicants/applicants_grid.php:

> @@ -348,6 +348,12 @@ public function actionGetApplicants(){
>      $this->loadView('applicants_single/result');
>    }
>
> +  public function actionGetApplicationTags(){


Tags are already included in the application JS loaded by service.getCurrentApplication. Instead of creating a new action you should add a listTags to the Display JS class.

—
Reply to this email directly or view it on GitHub<https://github.com/Jazzee/Jazzee/pull/245/files#r6472379>.