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.

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

[ALS-5510] Update JUnit #267

Merged
merged 5 commits into from
Jan 3, 2024
Merged

[ALS-5510] Update JUnit #267

merged 5 commits into from
Jan 3, 2024

Conversation

JamesPeck
Copy link
Contributor

No description provided.

Gcolon021 and others added 3 commits December 18, 2023 15:20
…#263)

This update modifies the visualization-modal-view.js file, moving Plotly from the main module dependencies to a new loadPlotly method. This approach lazily loads the Plotly library only when the modal is opened, optimizing the loading process. Modifications also include necessary adjustments to the render method to incorporate the new loadPlotly functionality.
The order of operations has been altered in the landing.js file of the BioDataCatalyst UI. The current changes involve rendering the template right after the function is invoked rather than waiting until the end. This reordering will allow the loading icons to display until all ajax request are complete.
@JamesPeck JamesPeck added the dependencies Pull requests that update a dependency file label Dec 21, 2023
@JamesPeck JamesPeck changed the base branch from master to release/0008 December 21, 2023 15:01
@JamesPeck JamesPeck changed the base branch from release/0008 to master December 21, 2023 15:03
@JamesPeck JamesPeck changed the base branch from master to release/0008 January 3, 2024 13:42
@JamesPeck JamesPeck merged commit 859fb4f into release/0008 Jan 3, 2024
2 checks passed
@JamesPeck JamesPeck deleted the bump-junit branch January 3, 2024 20:18
Gcolon021 added a commit that referenced this pull request Jan 22, 2024
…t release (#276)

* [ALS-5510] Update JUnit  (#267)
* [ALS-5567] Replace hard-coded URLs with variables in not_authorized.hbs (#268)
* [ALS-5669] Update 7 bridges text and link (#269)
* [ALS-5474] Fix study tags showing as tags (#270)
* [ALS-5528] Add Terms of Service modal to help dropdown (#271)
* [ALS-5336] Fix bug where users were seeing the DCC study tag when they should not (#272)
* Remove virtual host configuration files (#264)
* [ALS-5589] Data Dashboard: align to header (#275)
* [ALS-1302] Update text to Freeze9 (#274)

---------

Co-authored-by: James <Jamestp19@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file ignore-for-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants