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

Restore ORCID attribute on hyrax user object #1096

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

crowesn
Copy link
Contributor

@crowesn crowesn commented Nov 1, 2023

Fixes #982

Restore user profile edit form fields and display for ORCID attribute on user object.

Changes proposed in this pull request:

  • Remove ORCID profile connector widget from profile edit view
  • Remove ORCID profile connector widget from profile show view
  • Restore hyrax user orcid attribute form and display fields
  • Adjust specs

@crowesn crowesn force-pushed the 982/orcid-form-field branch 5 times, most recently from 42794e2 to de0e646 Compare November 2, 2023 14:38
@crowesn crowesn changed the title Restore ORCID attribute on hyrax user object WIP Restore ORCID attribute on hyrax user object Nov 2, 2023
@crowesn crowesn changed the title WIP Restore ORCID attribute on hyrax user object Restore ORCID attribute on hyrax user object Nov 8, 2023
@Janell-Huyck Janell-Huyck self-assigned this Nov 8, 2023
end

it 'persists orcid id after editing' do
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Janell-Huyck this should fix it.

@crowesn crowesn force-pushed the 982/orcid-form-field branch 2 times, most recently from 4461d58 to f9d8513 Compare November 9, 2023 19:15
@Janell-Huyck
Copy link
Contributor

I am able to add an Orcid number in the profile page.
Screenshot 2023-11-13 at 12 42 36 PM

@Janell-Huyck
Copy link
Contributor

After saving the update, the link is correctly displayed:
Screenshot 2023-11-13 at 12 41 08 PM

@Janell-Huyck
Copy link
Contributor

View when I return to profile edit page:
Screenshot 2023-11-13 at 12 41 53 PM

@Janell-Huyck Janell-Huyck merged commit f2a6fb6 into develop Nov 13, 2023
2 checks passed
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.

Allow users to manually add their ORCID ID to their profile page
2 participants