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.

Sign up for GitHub

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

Jump to bottom

Sync: bring OntoPortal up-to-date with BioPortal releases 6.15.0 and onward #19

Merged
merged 37 commits into from
May 30, 2024
Merged

Sync: bring OntoPortal up-to-date with BioPortal releases 6.15.0 and onward #19

merged 37 commits into from
May 30, 2024

Conversation

jvendetti
Copy link
Collaborator

The purpose of this pull request is to merge BioPortal's master branch with OntoPortal's master branch to align OntoPortal with code from the last two BioPortal releases (versions 6.15.0 and 6.16.0).

To follow are slightly abbreviated release notes for the above-mentioned releases.

6.15.0

  • Upgrade Rails from 6.1.7.3 to 7.0.8
  • Bump ontologies_api_client from 2.2.4 to 2.2.5

6.16.0

  • Rails 7.0.8 upgrade (part two)
    • Install Turbo (though currently disabled)
    • Install Stimulus
    • Install jsbundling-rails with the esbuild bundler
    • Add the sprockets-rails gem and upgrade Sprockets from 3 to 4
    • Replace sass-rails with sassc-rails
    • Refactor the Gemfile to better mimic Rails 7.0.8 default gems
  • Fix issue that prevented the "Login as" functionality from working for account names with '@' symbols
  • Fix issue that prevented deletion of users with '@' symbols in the account name
  • Fix some RuboCop warnings
  • Bump copyright year

@jvendetti
Bump rails from 6.1.7.3 to 7.0.8
78aaa9b
@jvendetti
Result of app:update
dee0f62
@jvendetti
Result of db:migrate
8031913
@jvendetti
Add purl configuration settings for test mode
b290746 
Fixes undefined method errors when running RSpec tests
@jvendetti
Exit License model tests if appliance ID changes
d66b10b 
Adds a before all block to the License model RSpec tests to handle the rare case when the appliance ID of the BioPortal staging server changes.
@jvendetti
Fix broken License model unit tests
4244c62 
The appliance ID of the BioPortal staging server changed, resulting in a scenario where the license generated for testing purposes was no longer a match and caused test failures.
@jvendetti
Fix some RuboCop warnings
dd4a774
@jvendetti
Enable a subset of the new framework defaults
156ff9f
@jvendetti
Temporary pointer to ontologies_api_client branch
fb97874
@jvendetti
Bump ontologies_api_client to v2.2.5 tag
67aa1bd
@jvendetti
Remove old framework default files
e664bab
@jvendetti
Merge pull request #302 from ncbo/refactor/rails-7-upgrade/297
a5b130e 
Upgrade Rails from 6.1.7.3 to 7.0.8
@jvendetti
Bump copyright
a6ff806
@jvendetti
Fix undefined method error
35370ea 
Resolves #308
@jvendetti
Fix some RuboCop warnings
1d54018
@jvendetti
Replace sass-rails with sassc-rails
a200f3f
@jvendetti
Add sprockets-rails gem and update from 3 to 4
ef2098e
@jvendetti
Install jsbundling-rails
3fe990b
@jvendetti
Rename build script entry point
e9710fb 
Fixes "Sprockets::DoubleLinkError: Multiple files with the same output path cannot be linked ("application.js")"
@jvendetti
Update manifest.js
06ac94c
@jvendetti
Install Turbo
cd23fb4
@jvendetti
Install Stimulus
b4fda30
@jvendetti
Add include tags for JS bundled with esbuild
2c4685c
@jvendetti
Fix some RuboCop warnings
3cd25de
@jvendetti
Remove example Stimulus controller
edd87a9
@jvendetti
Add more Rails 7 default gems
54a3537
@jvendetti
Fix uninitialized constant error
44f3fe6
@jvendetti
Encode username component in login_as link
364a484 
Resolves #299
@jvendetti
Encode username component for deletion
1deef1f 
Fixes an issue where account deletion fails for account names with '@' symbols
@jvendetti
Merge pull request #309 from ncbo/refactor/rails-7-upgrade-continued
9b3ef6a 
Rails 7.0.8 upgrade (part two)
@syphax-bouazzouni syphax-bouazzouni merged commit 97384d1 into ontoportal:master May 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bilelkihal Bilelkihal Bilelkihal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jvendetti @Bilelkihal @syphax-bouazzouni