[KT] (AICCRA) Develop Feedback Functionality

[kenjitm]

Develop the feedback functionality in AICCRA according to the following template: https://docs.google.com/presentation/d/1AEW7EoDeNzOxVsoMZLDZqZoE9oHfWnA37_kXI0tuCx4/edit#slide=id.ge82d74f50f_0_5

• [x] It must be possible to enable or disable the functionality of commenting on a specific field
  

• [x] It must possible to comment on a certain field by the Module Leaders and Managers Module
  

• [x] It must be possible to respond to the comments, approve or disapprove them, by the Leaders and Coordinators of the cluster
  

• [x] A section should be created where the information of the comments made in the fields at the cluster level can be seen in a consolidated manner through the embedding of a BI report
  

• [x] It must be indicated next to the name of each section of the cluster, the number of pending comments to be answered.
  

• [x] At the administrator level, a section must be created in which a BI graph is embedded showing the comments consolidator for all clusters

• [x] Create specificity for safeguard section

• [x] Allow delete comments

[kenjitm]

It's possible to enable comments pop-up functionality for specific fields:

And save comments for the selected field:

[kenjitm]

• Feedback functionality is filtered by roles, cluster leaders and coordinators will be able to approve, reject or respond to comments.
• The comment left by a user is displayed along with its information

[MargaritaRamirez]

@valviveros In meeting with Hector this morning, he requests the following change in the feedback status banner

blue: No comments provided green: All comments tackled

[valviveros]

The requested change has been made.

[MargaritaRamirez]

In addition we will remove the icon at the right and colored the comments boxes

[yecksin]

About the May 26th meeting =>

It was discussed that there is no problem with the development time on the front end, since the MARLO logic is complex, so you must have a good level of abstraction and recursion. However, it can be improved by receiving adequate information in the services, e.g. that the "list" is iterable by default, since with many objects a more complex and time-consuming mapping must be done.

The only advice is to keep in mind good practices in "Clean Code" to be able to retake code when time goes by and reduce the technical debt in the future.

If "MARLO / AICCRA" is going to have more development time in the future with new features, it is better to resume the use of Vue.js as Sebastian did in the past. The advantage is also that you can hire people who work with Vue.js because we can reduce the use of MARCOS with FreeMarker and have a more dynamic code.

[MargaritaRamirez]

Message sent on June 8th, 2022

[MargaritaRamirez]

Replay from Laura (our comments in blue)