Skip to content

Fix share required params#2874

Merged
briri merged 2 commits intoDMPRoadmap:developmentfrom
nicolasfranck:fix_share_required_params
Apr 16, 2021
Merged

Fix share required params#2874
briri merged 2 commits intoDMPRoadmap:developmentfrom
nicolasfranck:fix_share_required_params

Conversation

@nicolasfranck
Copy link
Contributor

Changes proposed in this PR:

  • check on existance of :email in hash :user. If you do not supply an email address now, the controller crashes
  • make fields email and access required in html

Copy link
Contributor

@briri briri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good. Thank you @nicolasfranck!

@briri briri merged commit 72de423 into DMPRoadmap:development Apr 16, 2021
portagenetwork pushed a commit to portagenetwork/roadmap that referenced this pull request Feb 24, 2022
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.

2 participants