Feature/integration model#504
Open
ivanmrsulja wants to merge 3 commits into
Open
Conversation
chenejac
requested changes
May 3, 2026
Comment on lines
+546
to
+554
|
|
||
| .mt-5 { | ||
| margin-top: 3.5rem; | ||
| } | ||
|
|
||
| .section-title-small { | ||
| margin-bottom: 0rem !important; | ||
| } | ||
|
|
Contributor
There was a problem hiding this comment.
What is the purpose of those lines? Why !important? Can that be changed in themes if it is defined as here? Morover, ORCID integration is VIVO specific task, not sure why css configuration is in Vitro code.
Member
Author
There was a problem hiding this comment.
I consulted @milospp and he suggested to move this CSS to VIVO's wilma.css, so I did that. Also, !important is not needed anymore because, after moving the code, load order is as it should be.
ivanmrsulja
force-pushed
the
feature/integration-model
branch
from
June 15, 2026 06:55
a013710 to
a7f782a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issues: 4074, 4075
linked VIVO PR: 4125
What does this pull request do?
Added support for integration model graph, where integration metadata (access tokens, push history, utility metadata) can be stored for various external platforms.
What's new?
How should this be tested?
This should be tested in conjunction with corresponding VIVO PR that introduces Orcid Member API integration for push workflows. It uses this model for integration metadata storage, if it works as expected, so does this.
Interested parties
@chenejac
Reviewers' expertise
Candidates for reviewing this PR should have some of the following expertises: