Skip to content

Scheduler-UI-NEXT from develop to master Code push regarding CI Build Configurations and some fixes#39

Merged
helenKaryamsetty merged 24 commits into
masterfrom
develop
Jul 1, 2024
Merged

Scheduler-UI-NEXT from develop to master Code push regarding CI Build Configurations and some fixes#39
helenKaryamsetty merged 24 commits into
masterfrom
develop

Conversation

@bhpratibha
Copy link
Copy Markdown
Contributor

Scheduler-UI-NEXT from develop to master Code push regarding CI Build Configurations and some fixes

const SCHEDULER_API = '<%= SCHEDULER_API_BASE %>';
const TM_API = '<%= TM_API_BASE %>';

const MMU_API = `http://${IP}:8080/mmuapi-v1.0/`;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't look right.
http and port numbers shouldn't be here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Even the version of the mmu API

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@drtechie HI Mithun, updated as per comments, Please check

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
16.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Copy Markdown
Member

@helenKaryamsetty helenKaryamsetty left a comment

Choose a reason for hiding this comment

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

looks fine

@helenKaryamsetty helenKaryamsetty merged commit d7c01a8 into master Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants