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

improve update summary count calculations (DAT-16656 and DAT-16357) #5634

Merged
merged 10 commits into from Mar 13, 2024

Conversation

StevenMassaro
Copy link
Contributor

Impact

  • Bug fix (non-breaking change which fixes expected existing functionality)
  • Enhancement/New feature (adds functionality without impacting existing logic)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

@MalloD12 MalloD12 changed the title improve update summary count calculations (DAT-16656) improve update summary count calculations (DAT-16656 and DAT-16357) Mar 1, 2024
@MalloD12
Copy link
Contributor

MalloD12 commented Mar 1, 2024

Guys, I just pushed a minor change to update "Total change sets" calculation in case you want to look at it.

@StevenMassaro
Copy link
Contributor Author

Guys, I just pushed a minor change to update "Total change sets" calculation in case you want to look at it.

@MalloD12 Looks good

@MalloD12
Copy link
Contributor

Again, added another minor change to include "Failed deployment" changesets count to the total. Made a quick manual test in Core and Pro and it looks good to me.

@StevenMassaro StevenMassaro merged commit 5f5624e into master Mar 13, 2024
31 of 34 checks passed
@StevenMassaro StevenMassaro deleted the DAT-16656 branch March 13, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants