DMPbelgium / roadmap

5 stars 1 forks source link

duplicate org name in guidance choices #87

Open StCyr opened 1 year ago

StCyr commented 1 year ago

Please complete the following fields as applicable:

What version of the DMPRoadmap code are you running? (e.g. v2.2.0)

4.1.0

Expected behaviour:

In the screenshot in the "actual behaviour" chapter below, Belnet should appear only once

Actual behaviour:

duplicate org name in guidance choices.

In the screenshot hereunder, Belnet appears twice:

image

Possible fix

change:

<% choices.each do |org, groups| %>

to

<% choices.uniq.each do |org, groups| %>

in app/views/plans/_guidance_choices.html.erb

nicolasfranck commented 1 year ago

@StCyr better to update @important_ggs in app/views/plans_controller.rb#show.

What happens: in the "short" view of available guidance groups it shows the following:

And in this case "Belnet" is both a default organization, and you happen to be attached to organization Belnet.

BTW: in the full view (via the modal) it is all fine.

StCyr commented 1 year ago

@TheLisaVL and I are wondering if it wouldn't be better to set "DCC" as default organisation...

nicolasfranck commented 1 year ago

Forgot about the rationale behind the default organization. In the code I only see references to Org.default_orgs in code that handle guidances and guidance groups. So maybe the default organization is being misused?

@laurastandaert @mm449 ?

StCyr commented 5 months ago

should be fixed in version 4.2.0