-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Project editor refactor and RSR / IATI split #1895
Comments
Background Decisions:
|
four questions for @lauraroverts and @KasperBrandt
|
My two main question right now is how to differentiate between the different mandatory fields. How do we show what fields influence each of the progress bars? RSR mandatory fields are indicated with a red *. So far the only thing I can think of is to indicate the other mandatory fields with a different color *, but @lauraroverts thinks this will get confusing. Also; we now show a progress bar in every section header. There's no room for more than 1 progress bar there. So it will be hard to make the same differentiation per section. |
The worst choice for "IATI mandatory" fields would be a green *. (Lots of colour-blind people out there.) There are other symbols, like § ▲ ⊙ ¹ . |
I have been looking at https://docs.google.com/spreadsheets/d/1iwnT5XIdmxhVKJfenF03jjyDbdyJV5oj19O1M-EJZ-Y/edit#gid=0, and I have a few questions about that:
|
@KasperBrandt, @lauraroverts |
@KasperBrandt, @lauraroverts, a proposal for the requirements:
|
Regarding how to structure the editor: I propose: If you add a new project, you first have to indicate if you are going to publish your project to RSR or 'RSR + IATI'. (It was decided to skip publishing to IATI without RSR.) So depending on what you choose here, the Project Editor shows you the relevant fields. If you publish to RSR, you won't see IATI identifier for example. RSR + IATI shows all fields. |
That is an interesting option, which brings other questions to mind though:
|
@lauraroverts, ok, clear. |
@mtwestra Hm, yes, that's what I was thinking too. That leaves us with the same issue on progress bars though. If we show one for RSR (if completed, it's possible to publish project in RSR) and one for IATI (if completed, it's possible to publish to IATI) on a 'RSR + IATI' project, what would be wise in order to distinguish the RSR or IATI mandatory fields? |
Why distinguish? If you choose to publish to RSR + IATI, there will be just a few more fields with a red asterix. If there is one progress bar, the bar knows which fields are mandatory, so you can easily see when it is ready to publish to RSR + IATI. If you choose to publish to RSR, you won't see these IATI mandatory fields. |
In the case of an RSR only project, the situation is clear. Say we have and RSR+IATI project, in which all mandatory RSR fields have been filled in, but not all mandatory IATI fields. What should we show? If we use a 'combined' progress bar, the user might conclude that the project cannot be published to RSR, while that is not the case. |
Decision regarding progress bars when selecting RSR + IATI:
|
I would probably show both overall progress bars at the same time - it is quite clear I think. |
Yup, this makes sense. Another question; where would we set if a project is RSR or RSR + IATI? And who has the permission to set or change this? |
The data migration 0045 won't work when pk=1 is included when creating the first object. At least on my vagrant box I get a unique constraint error, so I'm removing the setting of the pk. This means it's imperative to run migration 0044 and 0045 at the same time to guarantee that the RSR validation set gets pk=1.
[#1895] Project editor refactor and validation sets
Before we can request the partnerteam and partners to test the new functionality, I've got a few comments that we need to discuss. If I look at http://rsr.test.akvo.org/en/myrsr/project_editor/4236/, then:
|
Use the values for ProjectEditorValidation.MANDATORY_ACTION and ProjectEditorValidation.HIDDEN_ACTION when creating validation objects.
Add PublicProjectViewSet.project_relation field used in get_queryset() Refactor PublicProjectViewSet.get_queryset(). The new code removes the need for get_queryset() in sub-classes to filter out non-public projects, instead relying on overriding the value of PublicProjectViewSet.project_relation.
Since we've got some time before next release, I'm wondering what to do now?
|
[#1895] One progress bar in project editor Code reviewed
Test plan
As this is a large change (a complete refactor) of the project editor, it should be tested thoroughly. The following points should definitely be tested:
Editor basics:
Special cases:
Validation sets and progress bars:
Issue description
For an overview of the discussion around the RSR and IATI editor split, see this document.
The text was updated successfully, but these errors were encountered: