Ну, как бы вроде даже и все.

Идею положить в sidebar и другие проекты откладываем в другой релиз в задачу tndt-145

Вопрос можно ли проверить полномочие не текущего проекта, хорошо ли, что в voter приходит текущий проект, как это сделать правильнее и понятнее оставили на потом в tndt-144

Идее отображать в SidebarMenu еще несколько проектов нам помешает PrivateProjectVoter, он зависит от ProjectContext, поэтому по факту является CurrentProjectVoter, т.е. он проверяет полномочие пользователя относительно текущего проекта, а не указанного, а значит в нашем Security нельзя сделать проверку isGranted(PROJECT_SETTING, $project). Это надо будет вынести в отдельную задачу в другой релиз в рамках рефакторинга Voter’ов

Ну вот, самую тяжелую часть вынесли. Почистили от самых ненужных пунктов, остальные вроде бы соорганизовали более менее. Надо теперь NavBar так же перенести, чтобы он не исчез по отключению AdminLte.

И подумать насчет или выводить в sideBar не только текущий проект, или не делать проект двух-уровневым. (кажется для разных юскейсов первый вариант интереснее)

Как-то покаместь и для sidebar убрали index, вроде бы приоритет сабскрайберов вполне справляется, а внутри сабскрайбера он сам определяет порядок.

интересное дело, в breadcrumb важен порядок вызова листенеров, и совсем не нужен id.

В SideMenu тоже скорее всего каждому листенеру захочется воткнуть свой пункт как-то по своему, например общие пункты могут появиться в начала и конце. (хотя все проекты, это вроде про проекты)

В итоге не отложили, и лишний вечер потратили на него. Ну и хлебные крошки доделали.

Или не откладывать, а сделать параллельно tndt-39, да это еще раздует релиз, но вроде бы это быстро.

Правда вопрос, Нужна ли на странице редактирования задачи хлебная крошка с именем задачи дискуссионен. (Например в Jira Confluence она есть)

Может быть и не бесполезна, чтобы в неё вернуться. Но можно и отложить это, чтобы не делать лишнего запроса.

В целом в процессе переноса можно легко избавляться от пунктов, которые нам не слишком нужны. Когда мы поймем, что нам не хватает тут его, добавим обратно. Плюс, меньше пунктов, поможет нам перенести содержимое sidebar в navbar

В рамках переноса, пока нет необходимости упоминать в хлебных крошках страницы редактирования задачи, может и не делать этого? Когда мы поймем, что в крошках очень не хватает названия задачи/документа, тогда и будем тратить запрос на заполнение. Может к тому времени у нас они будут в реквест класться tndt-39, и тога их будет достаточно от туда вынуть.

У нас пока не устаканилось дерево подписчиков

Вариант 1

• Breadcrumbs
  • CommonItems
  • ProjectItems
  • TaskItems
  • DocItems
• Sidebar
  • CommonItems
  • ProjectItems
  • TaskItems
  • DocItems
• Navbar
  • CommonItems
  • ProjectItems
  • TaskItems
  • DocItems

Вариант 2

• Common
  • BreadcrumbsSubscriber
  • SidebarSubscriber
  • NavbarSubscriber
• Project
  • BreadcrumbsSubscriber
  • SidebarSubscriber
  • NavbarSubscriber
• Task
  • BreadcrumbsSubscriber
  • SidebarSubscriber
  • NavbarSubscriber
• Doc
  • BreadcrumbsSubscriber
  • SidebarSubscriber
  • NavbarSubscriber

Кроме того, сборка сайд-меню и хлебных крошек столь похожи, что хочется объединить часть сервисов в один. Да, для этого мы их хотели разъединить

Пока не сделано tndt-85 как-то малоосмысленно делать отдельную директорию resentation. Пусть все эти меню будут в корне, не страшно. Потом переместим сразу в Presentation/Event/…Bar