Skip to content
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

V3.5.11 beta #361

Merged
merged 33 commits into from
Aug 17, 2022
Merged

V3.5.11 beta #361

merged 33 commits into from
Aug 17, 2022

Conversation

briri
Copy link
Collaborator

@briri briri commented Aug 16, 2022

Changes proposed in this PR:

  • Comment out the institutional sso section of the 3rd party apps page. The login workflow now guides users to SSO automatically. Allowing them to disconnect here has proven to be problematic as it then removes their ability to log back in. We will likely re-enable this once we have merged in the sign-in-without-sso branch which allows users to bypass SSO and use a password instead.
  • Fix for DMPRoadmap #3199 that prevents an org admin from making all guidance groups and optional_subset. This scenario was causing 500 level errors because optional subsets are treated as children of the parent non-optional_subset group.
  • Fix for re3data repos that have multiple <repositoryContact> elements so that the email address is used
  • Fixed an issue that was preventing some plans from appearing on the public plans page if 100% of the questions had not been answered DMPRoadmap #3086
  • Fix for API v1 that was returning duplicate records DMPRoadmap #3200
  • Fix for an issue displaying metadata standards if one of the records did not contain a location DMPRoadmap PR#3171
  • Updated gem and JS dependencies
  • Adjustments to account for new Rubocop rules

This update requires bin/rails external_api:load_re3data_repos to be run to fix the contact info in the repositories table

briri and others added 30 commits April 12, 2022 09:38
fix issue where address lines that include a comma are interpretted a…
fix issue with logos disappearing when other Org data is saved
fix issue with string token for model save errors
one could only share plan if it is 100% completed. When this is set in
property config/initializers/_dmproadmap.rb, e.g.,
188:    config.x.plans.default_percentage_answered = 50.

Fix for Roadmap bug DMPRoadmap#3086.

In app/models/phase.rb method visibility_allowed?(plan) is broken
because the first line in method always return 0 unless the Rational()
is 1 in
  value = Rational(num_answered_questions(plan), plan.num_questions) * 100

Change:
- we fix issue by converting the Rational() value to a float as follows
    value = Rational(num_answered_questions(plan), plan.num_questions).to_f * 100
…rg_visibility_plan_public_DMPs_only_appear_if_100_per_filled

Roadmap bug 3086 - The share feature for Org plan visibility was broken:
…arch_result

metadata standard does not always have locations
Update master prior to v3.1.1

(brakeman fails on end of life warning, so ignoring.)
…nts' page not allowing downloads unless the user is signed in
Fixes an issue preventing users from downloading templates from the Funder Requirements page
…ns duplicate plans in some cases.

Fix for issue DMPRoadmap#3200.

Changes in app/policies/api/v1/plans_policy.rb methods
plans_for_client() and plans_for_user() returns a unique array of plan
ids by finally calling the method uniq() on the ids the methods
previously returned.
…ll_get_plans_returns_duplicates

Bug DMPRoadmap#3200 - Fix for the V1 API call GET /api/v1/plans returns duplica…
updated dependencies and added brakeman exception for EOL rails
… groups optional subsets

updates to prevent deletion of only non-optional subset

tweak to logic when editing guidance_group

tweak to logic when editing guidance_group

tweak to logic when editing guidance_group

tweak to logic when editing guidance_group

added @guidance_groups to update and create so they are available during view render
@briri briri merged commit 6063f6e into main Aug 17, 2022
@briri briri deleted the v3.5.11-beta branch August 17, 2022 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants