...
Agenda
- VIVO 1.15.0
- To be reverted
- Active
- I forgot my password
- Role based search filters
- https://github.com/vivo-project/Vitro/pull/442
- Reviewers needed!
- Profile linking with Matching ID doesn't work
- https://github.com/vivo-project/VIVO/pull/3947
- https://github.com/vivo-project/Vitro/pull/444
- The second reviewer needed
- SPARQL authorization cache
- https://github.com/vivo-project/Vitro/pull/450
- The second reviewer needed?
- Object property statement can't be hidden from display without hiding all property statements in individual profile
- https://github.com/vivo-project/Vitro/pull/451
- The second reviewer needed?
- Menu pages can't be modified
- Date time interval doesn't allow end date to be equal start date
- https://github.com/vivo-project/Vitro/pull/453
- Ready to be merged
- New
- Research areas
- Readme file, contribution, join the community
- contributing.md
- screenshots?
Notes
...
Responsive wilma theme
Team decided to revert PRs related to making the Wilma theme responsive, to create a new one PR which will fix some meanwhile noticed issues with this theme.
The PR https://github.com/vivo-project/VIVO/pull/3932 was reverted by using a GitHub option.
Due to some conflict, PRs https://github.com/vivo-project/VIVO/pull/3909, https://github.com/vivo-project/Vitro/pull/429 could not be automatically reverted by using GitHub. Georgy will try to make revert PRs for those. Dragan will review and merge those PRs created by Georgy. Dragan will inform Milos (the author of PRs) to cherry pick commits and create new PRs (one for VIVO and one for Vitro). The next steps will be to test it and to fix issues.
I forgot my password
https://github.com/vivo-project/VIVO/pull/3916
https://github.com/vivo-project/Vitro/pull/421
Georgy and William will review this PRs pair. Ivan reported that some i18n messages have been changed, Dragan suggested validating those messages by native speakers after reviewing PRs by Georgy and William.
Role based search filters
https://github.com/vivo-project/Vitro/pull/442
The second reviewer for this PR will be William. In the case, William can’t find time to review this in the next couple of weeks, an alternative reviewer might be Ivan.
Profile linking with Matching ID doesn't work
https://github.com/vivo-project/VIVO/pull/3947
https://github.com/vivo-project/Vitro/pull/444
Dragan will review this again. However, a second reviewer is needed for reviewing the code of this PR for sure.
SPARQL authorization cache
https://github.com/vivo-project/Vitro/pull/450
Dragan thinks the second reviewer for this PR is not needed and will suggest merging of this PR with one reviewer in the #committers channel. If there is no objection, it might be merged (NOTE 1: only the last commit is new in this PR, which changed one java file and adding a JUnit test
NOTE 2: this PR has to be merged after merging https://github.com/vivo-project/Vitro/pull/444)
Object property statement can't be hidden from display without hiding all property statements in individual profile
https://github.com/vivo-project/Vitro/pull/451
Dragan thinks the second reviewer for this PR is not needed and will suggest merging of this PR with one reviewer in the #committers channel. If there is no objection, it might be merged.
Menu pages can't be modified
https://github.com/vivo-project/Vitro/pull/452
Dragan tested this PR, after reviewing the code he will report back to Georgy (author of the PR)
Date time interval doesn't allow end date to be equal start date
https://github.com/vivo-project/Vitro/pull/453
Dragan thinks the second reviewer for this PR is not needed and will suggest merging of this PR with one reviewer in the #committers channel. If there is no objection, it might be merged.
TinyMCE
https://github.com/vivo-project/Vitro/pull/454
Dragan will test and review the code.
JQuery
https://github.com/vivo-project/VIVO/pull/3955
https://github.com/vivo-project/Vitro/pull/449
This PR is complicated for testing and reviewing. Benjamin explained the process he applied in the implementation of this PR. It is already at a development server for one VIVO instance, meaning it is at least partially tested. The issue is whether it is working for all themes. Georgy suggested to split PR into:
- A PR which is adding a newJQuery directory with new library (and keeping old jquery library). This PR can be safely and easily merged in the main branch
- A separated PRs for each theme
At the end, we can in one minor VIVO release announce that usage of old jquery is deprecated, and in the next minor VIVO release we can remove old jquery.
Benjamin switched PR to draft, and he will wait to inform him once responsive wilma theme PR is merged to start making PRs.
Research areas
Georgy explained there is the option in Researchers profile pages (or Work pages) to manage research areas, and there is the option to create new or select an existing concept. Dragan will reply to the email with this explanation.
Actions
- Georgy Litvinov to create revert PRs for wilma responsive PRs
- Dragan Ivanovic to review (and merge if it ok) revert PRs
- William Welling to review assigned PRs
- Dragan Ivanovic to respond on email in accordance with discussion at this meeting
- Benjamin Gross to split JQuery PR into smaller one (one per theme) once when "Wilma responsive theme" task is completed and merged
- Dragan Ivanovic to discuss merging of simple PRs with only one reviewer in the #committers slack channel
Previous actions
- Dragan Ivanovic to prepare VIVO 1.14.1 release candidate
- Dragan Ivanovic to post a message in the VIVO committers channel about questions from this meeting
- Dragan Ivanovic to align Pull request template with Brian's suggestions
- Dragan Ivanovic to draft CONTRIBUTING.md file
- Dragan Ivanovic to create a new ticket for log4j vulnerability mitigation and to link to https://github.com/vivo-project/Vitro/pull/441
- William Welling to review https://github.com/vivo-project/Vitro/pull/421 and https://github.com/vivo-project/VIVO/pull/3916
- Georgy Litvinov to draft updates for https://github.com/vivo-project/VIVO/wiki/Development-Processes#process-for-reviewing-contribution
- Dragan Ivanovic to draft updates for PR template
- Dragan Ivanovic to contact Arif regarding opening a ticket (feature request) for home page statistics
- Dragan Ivanovic will inform Ivan about decisions for I forgot my password PR
- Georgy Litvinov to review https://github.com/vivo-project/Vitro/pull/427
- Brian Lowe to review https://github.com/vivo-project/Vitro/pull/424
- Georgy Litvinov to improve https://github.com/vivo-project/Vitro/pull/433 (UI options for defining policies)
- Dragan Ivanovic to work on wiki documentation
- Dragan Ivanovic to contact Ralph O'Flinn about https://github.com/vivo-community/create-wsl2-vivo-dev-env
- Dragan Ivanovic to communicate with the Lyrasis team about adding commenting permissions to anonymous wiki users
- Ivan Mrsulja to implement suggested improvements into https://github.com/vivo-project/Vitro/pull/427
- Georgy Litvinov to improve https://github.com/vivo-project/Vitro/pull/390 in accordance with Miloš Popović review
- Dragan Ivanovic to review https://github.com/vivo-project/Vitro/pull/431
- Dragan Ivanovic to review https://github.com/vivo-project/Vitro/pull/432
- Georgy Litvinov to complete https://github.com/vivo-project/Vitro/pull/433
- Dragan Ivanovic to discuss with Milos Popovic about usage of JQuery in the nemo theme
- Georgy Litvinov to provide Dragan the information about volume size needed for mirroring GitHub into GitLab
- Dragan Ivanovic and Georgy Litvinov to complete merging the ABAC PRs
- Dragan Ivanovic to review extended search PR and Avoid lock on graph uri update in RdfServiceSparql PR
- William Welling will continue working on vivo-solr upgrade branch
- Ivan Mrsulja will consider adding a configuration flag for turning on/off the "I forgot my password" feature
...