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

Reload workspace options on paymodel change #1447

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

nss10
Copy link
Contributor

@nss10 nss10 commented Nov 1, 2023

Jira Ticket: BRH-486

Bug Fixes

  • Reload Workspace Options when a user selects a different pay model

Copy link
Contributor

@paulineribeyre paulineribeyre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome

@nss10 nss10 merged commit 4eb54b0 into master Nov 6, 2023
@nss10 nss10 deleted the fix/reload_ws_options branch November 6, 2023 15:14
jarvisraymond-uchicago added a commit that referenced this pull request Nov 14, 2023
* feat: allow teamProject to be configured in gitops.json instead of hard-coding in frontend code

* feat: allow for appId property in app definition

...and if set, use that instead of "title"

* feat: added analysisTools section documentation to portal_config.md

* fix: revert back the old GWAS app names

...so navigation can work again as before

* feat(addTeamProjecHeaderToAnalysisApps): Added header without logic to redirect

* feat(addTeamProjectHeaderToAnalysisApps): added redirect logic

* feat(addTeamProjectHeaderToAnalysisApps): Got test to work for TeamProjectHeader conditional

* feat(addTeamProjectHeaderToAnalysisApps): Ran eslint-new

* feat(addTeamProjectHeaderToAnalysisApps): Removed unneeded async declaration

* feat(addTeamProjectHeaderToAnalysisApps): Removed unneeded waitFor import

* feat(addTeamProjectHeaderToAnalysisApps): Fixed lint issues

* feat(addTeamProjectHeaderToAnalysisApps): Removed header element to make styles more consistent across applications

* feat(addTeamProjectHeaderToAnalysisApps): Removed unneeded comments

* feat(addTeamProjectHeaderToAnalysisApps): Ran linter

* feat(addTeamProjectHeaderToAnalysisApps): Changed all instances of variable name showButton to isEditable

* feat(addTeamProjectHeaderToAnalysisApps): updated storybook for clarity

* Reload workspace options on paymodel change (#1447)

* feat(teamProjectApiUpdate): Initial commit

* feat(teamProjectApiUpdate): Added API update for results app

* Chore(deps-dev): Bump @babel/traverse from 7.22.5 to 7.23.2 (#1440)

Bumps [@babel/traverse](https://github.com/babel/babel/tree/HEAD/packages/babel-traverse) from 7.22.5 to 7.23.2.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.23.2/packages/babel-traverse)

---
updated-dependencies:
- dependency-name: "@babel/traverse"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(teamProjectApiUpdate): Added conditional in AnalysisApp.jsx for OHDSI Atlas app iframe URL

* feat(teamProjectApiUpdate): Refactored initializeCurrentState for terseness

* feat(teamProjectApiUpdate): updated API endpoint per change in BE

* feat(teamProjectApiUpdate): updated data passed to HomeTable component to support change in BE response

* feat(teamProjectApiUpdate): updated data passed to HomeTable test to support change in BE response

* feat(teamProjectApiUpdate): Reverted change on FE to deal with BE change in DS, BE change was inadvertant

* feat(teamProjectApiUpdate): Reverted unintentionally changed files

* feat(teamProjectApiUpdate): Removed unneeded console log statement

* feat(teamProjectApiUpdate): Changed OHSDI iframe src string based on discussed with BE dev

* feat(vadc_sprint22): Added sanitize function for team project variable

* feat(vadc_sprint22): Added const var for team project variable

* feat(vadc_sprint22): Refactored to use class function getAtlastURLWithTeamProject

* feat(vadc_sprint22): Autoformatted code

* feat(vadc_sprint22): Updated regex and test for regex

* feat(vadc_sprint22): formatted Code for linter

* feat(vadc_sprint22): Updated regex and autoformatted code

* feat(vadc_sprint22): Reverted changes for snyk

* feat(vadc_sprint22): updated regex and added throw error

* fix: let teamproject be added to Atlas only if the needsTeamProject flag is set

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: pieterlukasse <[email protected]>
Co-authored-by: Sai Shanmukha Narumanchi <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants