Updates in details of DF #54
Conversation
| @Autowired | ||
| @Qualifier("roleRepository") |
There was a problem hiding this comment.
@joaoeudes7 o que muda com essa alteração de Anotação?
There was a problem hiding this comment.
Como a instância já existe antes de chamar esse controller, é feito a alteração para o Qualifier, com isso ele não vai precisar ser recriado e precisa apenas chamar pelo nome
| // @GetMapping("/df/list") | ||
| // public ModelAndView list() { | ||
| // @GetMapping("/df/list/{projectId}") | ||
| // public ModelAndView list(@PathVariable("projectId") Long projectId) throws BusinessRuleException { | ||
| // ModelAndView mv = new ModelAndView("df/list"); | ||
| // | ||
| // User current = getCurrentUser(); | ||
| // User currentUser = getCurrentUser(); | ||
| // Project project = projectService.findOne(projectId); | ||
| // | ||
| // if (!project.isMemberOfProject(currentUser)) { | ||
| // throw MEMBER_NOT_EXISTS; | ||
| // } | ||
| // | ||
| // List<Project> projects = projectService.findByUserId(current.getId()); | ||
| // List<DataFunction> dfs = dataFunctionService.findByProject(projectId); | ||
| // | ||
| // mv.addObject("projects", projects); | ||
| // mv.addObject("project", project); | ||
| // mv.addObject("dfs", dfs); |
There was a problem hiding this comment.
@joaoeudes7 muito código comentado. Ainda está em construção?
There was a problem hiding this comment.
Para saber se posso remover, como os DFs vão ser puxadas a partir da view do projeto, não vejo mais necessidade dessa view de list, até por isso descartei o que eu havia feito, veja os detalhes da tarefas e #31
|
|
||
| // Consultar se precisa ser por projeto ou time a busca | ||
| @GetMapping("/df/list") | ||
| public ModelAndView list() { | ||
| ModelAndView mv = new ModelAndView("df/list"); | ||
|
|
||
| List<DataFunction> dfs = dataFunctionService.findAll(); | ||
|
|
||
| mv.addObject("dfs", dfs); | ||
|
|
||
| return mv; | ||
| } | ||
|
|
There was a problem hiding this comment.
@joaoeudes7 não tem mais o listar de DataFunction?
There was a problem hiding this comment.
Veja a resposta no outro comentário
joaoeudes7
force-pushed
the
feat/create_list_and_details_df
branch
from
d9cfbd9 to
457bd3e
Compare
| <script type="text/javascript"> | ||
| function viewDetailsDf(id) { | ||
| window.location = `details/${id}`; | ||
| } | ||
| </script> |
There was a problem hiding this comment.
O que acha de invés de adicionar um botão na linha da tabela, fazer a mudança de rota por meio do clique na linha? Utilizando JS
joaoeudes7
force-pushed
the
feat/create_list_and_details_df
branch
from
457bd3e to
c55deae
Compare
joaoeudes7
changed the title
joaoeudes7
force-pushed
the
feat/create_list_and_details_df
branch
from
c55deae to
b93f5e0
Compare
joaoeudes7
force-pushed
the
feat/create_list_and_details_df
branch
from
b93f5e0 to
f37da80
Compare
No description provided.