AjuntamentdeBarcelona / decidim-internal-evaluation-module

GNU Affero General Public License v3.0
0 stars 0 forks source link

New feature: Internal evaluation module #2

Open NilHomedes opened 2 months ago

NilHomedes commented 2 months ago

Is your feature request related to a problem? Please describe When a proposal is being reviewed by a group of evaluators, the current interface is not usable to discuss and make a collective decision on the answer of a proposal.

In that sense, it’s important to have the possibility to add an internal evaluation attached to a text to expone your assessment.

Describe the solution you'd like As an evaluator, I want to have a new feature that allows me to see clearly the evaluation made by others and to discuss collectively the answer to a proposal. This feature must comply with the following characteristics:

Glossary Internal evaluation status: This is the assessment made by the evaluator based on the proposal answer statuses, and is linked to a text field. This is not public. It is only for evaluators purposes. This is editable. Internal evaluation: This is a field composed by the internal status + a text field to justify your assessment. This will be editable but does not allow replies. This is what will be taken into account to define whether an evaluator has evaluated a proposal or not. Answers: This is the public answer to a proposal. Private notes: These are the comment space that evaluators use to discuss the answer to a proposal. This is not public. It is only for valuators purposes.

Mockups (needed)

Does this issue could impact on users private data? No

Acceptance Criteria:

furilo commented 2 months ago

What are the statuses? Are they fixed?

furilo commented 2 months ago

cc @NilHomedes

NilHomedes commented 2 months ago

What are the statuses? Are they fixed?

I'm referring to proposal answer statuses: https://decidim-lot2.populate.tools/admin/participatory_processes/branch-highway/components/4/manage/proposal_states After this PR https://github.com/decidim/decidim/pull/12052 , admins can define different proposal statuses to answer proposals.

furilo commented 2 months ago

Ok, it was not clear to me that they were the same.

furilo commented 2 months ago

Hi @NilHomedes

This is a first exploration of how this screen could look like:

  1. Collapsible box for Evaluations as we have for Private notes
  2. List of evaluations. I've combined date (last updated) and evaluator name in 1 column because we don't have enough space for one column for each
  3. Column for status + the text of the evaluation
  4. Notice of how many evaluations are done, how many pending (ie. 3 out 4)
  5. Action: you'd only be able to edit yours
57741

To decide: one a evaluator clicks on edit she will see the textarea.

What do you prefer?

Anything I'm missing?

furilo commented 2 months ago

(@NilHomedes please share with the rest of Product if needed, as I can't mention them from this repo)

furilo commented 2 months ago

@NilHomedes any feedback on this? Can we start? We are currently waiting for the OK as, as we spoke, this should be done first.

NilHomedes commented 2 months ago

Hi @furilo We have been discussing this yesterday with Barcelona Team, some comments:

To decide: one a evaluator clicks on edit she will see the textarea.

We think the best approach here could be to open a pop-up with the text field and the status selector. So the valuator has to write their assessment and select a status. If an evaluator want to edit their internal evaluation, they can click on the edit icon and edit it from the pop-up.

Order of the sections We want to have the "Private notes" section above the "Internal evaluations" section. The "Internal evaluations" section should be displayed by default. So the evaluators can easily see where they have to put their input.

furilo commented 2 months ago

We think the best approach here could be to open a pop-up with the text field and the status selector. So the valuator has to write their assessment and select a status. If an evaluator want to edit their internal evaluation, they can click on the edit icon and edit it from the pop-up.

OK

We want to have the "Private notes" section above the "Internal evaluations" section.

OK

The "Internal evaluations" section should be displayed by default. So the evaluators can easily see where they have to put their input.

What do you mean? That it's always open when you into a proposal? It's the current behaviour, if I'm not wrong.

NilHomedes commented 2 months ago

What do you mean? That it's always open when you into a proposal? It's the current behaviour, if I'm not wrong.

I mean that it would be open always be default. So, when an evaluator go into a proposal, they should see the "Internal Evaluation" field open: image I was saying that to differentiate the behaviour of "Internal evaluation" from "Private notes", which are collpased by default.

furilo commented 2 months ago

In my case, whenever I go into a proposal, the Private Notes are opened, even if I collapsed it in the previous page view.

furilo commented 2 months ago

Proposal page with Private Notes block before Internal Evaluations

51545

Modal to edit

13091
furilo commented 2 months ago

(take into account that currently Private Notes block will be opened by default)

NilHomedes commented 2 months ago

Thanks for the mock-up @furilo! Looks great. Just for the record, we have agreed that:

furilo commented 2 months ago

Right now Private note are collpased by default, this should remain as it is (see how works in Nightly)

I don't know where the change of having Private Notes opened by default has taken place. Just to confirm: it will be closed in any context, not just in this module, right?

(having it closed when this module is activated while having it opened if the module is not activated is possible, but it introduces extra complexity)

NilHomedes commented 2 months ago

I don't know where the change of having Private Notes opened by default has taken place. Just to confirm: it will be closed in any context, not just in this module, right?

Yes, Private Notes will always be closed in any context. This behaviour might be introduced it here: https://github.com/decidim/decidim/pull/13017 I just left a comment to alert on this.

entantoencuanto commented 1 month ago

One question. General admins and participatory space admins can create private notes on proposals. Will they be able to create internal evaluations on proposals as well? If so I would create an evaluator role after creating the evaluation automatically to not break the counters of evaluators vs evaluations

NilHomedes commented 1 month ago

One question. General admins and participatory space admins can create private notes on proposals. Will they be able to create internal evaluations on proposals as well?

No, only valuators assigned to a proposal will be able to create an internal evaluation on that proposal. If an admin is assigned as valuator, then they will be able to create an internal evaluation.

furilo commented 1 month ago

@NilHomedes is this the correct order?

71509
NilHomedes commented 1 month ago

@furilo The order should be:

This order is following the logic order of these actions: First, you use Private notes to discuss or add extra information in a proposal. Second, you write your evaluation. And, finally, you answer publicly the proposal.

Crashillo commented 1 month ago

@NilHomedes could you please gimme push permission to this repo, please?

NilHomedes commented 1 month ago

@NilHomedes could you please gimme push permission to this repo, please?

Done!