Skip to content

user's ORCID iD links to ORCID instead of Profiles settings - #203

Merged
bklaing2 merged 1 commit into
masterfrom
fix-orcid-link
Jul 30, 2025
Merged

user's ORCID iD links to ORCID instead of Profiles settings#203
bklaing2 merged 1 commit into
masterfrom
fix-orcid-link

Conversation

@bklaing2

Copy link
Copy Markdown
Member

Purpose

Make user's ORCID iD in settings link to ORCID instead of back to the Profiles settings page

closes: Add github issue that originated this PR

Approach

Open Questions and Pre-Merge TODOs

Learning

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

  • New feature (non-breaking change which adds functionality)

  • Breaking change (fix or feature that would cause existing functionality to change)

Reviewer, please remember our guidelines:

  • Be humble in the language and feedback you give, ask don't tell.
  • Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
  • Offer suggestions on how to improve code e.g. simplification or expanding clarity.
  • Ensure you give reasons for the changes you are proposing.

@bklaing2 bklaing2 self-assigned this Jul 30, 2025
@bklaing2
bklaing2 merged commit fab7223 into master Jul 30, 2025
2 checks passed
@bklaing2
bklaing2 deleted the fix-orcid-link branch July 30, 2025 14:05
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