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

194581 - integrate s100 ess updatesince endpoint to scs stub #410

Merged

Conversation

MastekRohitChettri
Copy link
Collaborator

@MastekRohitChettri MastekRohitChettri commented Jan 3, 2025

This PR consists changes for PBI : 194581

  • Code changes added
  • Unit tests added
  • Functional tests added

@Arit14975 Arit14975 marked this pull request as ready for review January 8, 2025 10:56
@Arit14975 Arit14975 requested review from rockydevnet and a team as code owners January 8, 2025 10:56
@ukho-bot
Copy link

ukho-bot commented Jan 8, 2025

Snyk checks have failed. 2 issues have been found so far.

Icon Severity Issues
Critical 0
High 1
Medium 0
Low 1

code/snyk check is complete. 2 issues have been found. (View Details)

@Arit14975 Arit14975 marked this pull request as draft January 10, 2025 08:43
@Arit14975 Arit14975 marked this pull request as ready for review January 13, 2025 16:05
Copy link
Collaborator

@Nadeem14456 Nadeem14456 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@harshal11869 harshal11869 left a comment

Choose a reason for hiding this comment

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

Looks Good.

Copy link
Collaborator

@Mayuresh-Gawde Mayuresh-Gawde left a comment

Choose a reason for hiding this comment

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

LGTM from QA!!

Copy link
Collaborator

@jrippington jrippington left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@jrippington jrippington merged commit 5210ca7 into develop-s100 Jan 15, 2025
10 of 11 checks passed
@jrippington jrippington deleted the dev/194581-integrate-ess-s100-updatesince-to-scs-stub branch January 15, 2025 11:23
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.

6 participants