griffithlab / civic-server

Backend Server for CIViC Project
MIT License
39 stars 32 forks source link

Gracefully handle ORCID login when user hasn't specified a name #688

Closed acoffman closed 3 years ago

acoffman commented 3 years ago

Something seems to have changed in their API responses. This field was previously always present but in some cases it no longer is.

In the case when we have no family or given name, we'll fall back on the orcid itself as a new user's name.

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 915617750


Changes Missing Coverage Covered Lines Changed/Added Lines %
app/models/user_adaptors/orcid.rb 0 2 0.0%
app/api_helpers/orcid_api.rb 0 11 0.0%
<!-- Total: 0 13 0.0% -->
Files with Coverage Reduction New Missed Lines %
app/api_helpers/orcid_api.rb 1 0%
app/models/assertion.rb 1 69.89%
app/models/variant_group.rb 1 62.07%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 861915238: -0.01%
Covered Lines: 2130
Relevant Lines: 13383

💛 - Coveralls