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

Fixes #893 Deploying steps indexing. #907

Merged
merged 6 commits into from
Oct 14, 2022

Conversation

mike-winberry
Copy link
Contributor

Description

Fix deploy-utils createComponentStepMap step numbering. Removed unused constant POLL_TIME_MS. Added some commenting in deploy-utils.

Fixes # (issue)
#893

Type of change

  • Bug fix (non-breaking change which fixes an issue)

…d constant POLL_TIME_MS. Added some commenting in deploy-utils.
@mike-winberry mike-winberry self-assigned this Oct 13, 2022
@mike-winberry mike-winberry linked an issue Oct 13, 2022 that may be closed by this pull request
@mike-winberry
Copy link
Contributor Author

Screen.Recording.2022-10-13.at.12.21.17.PM.mov

@jeff-mccoy
Copy link
Contributor

thanks for adding videos on these PRs @mike-winberry, super helpful.

Copy link
Contributor

@jeff-mccoy jeff-mccoy left a comment

Choose a reason for hiding this comment

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

Was the POLL_TIME_MS just old code?

Copy link
Contributor

@Noxsios Noxsios left a comment

Choose a reason for hiding this comment

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

lgtm, if you wanted to make Typescript happier, change some lets to const as they do not get re-declared.

@mike-winberry
Copy link
Contributor Author

@jeff-mccoy Yes, that was old code and for both of these @Noxsios thanks I am sorely missing my pre-commit hooks, but should be cleaned up now.

@mike-winberry mike-winberry enabled auto-merge (squash) October 13, 2022 23:36
@mike-winberry mike-winberry merged commit 9a8f839 into master Oct 14, 2022
@mike-winberry mike-winberry deleted the 893-ui-bug-deploying-stepper-numbering branch October 14, 2022 04:19
Noxsios pushed a commit that referenced this pull request Mar 8, 2023
## Description

Fix deploy-utils createComponentStepMap step numbering. Removed unused
constant POLL_TIME_MS. Added some commenting in deploy-utils.

Fixes # (issue)
#893 


## Type of change


- [x] Bug fix (non-breaking change which fixes an issue)

Co-authored-by: Megamind <[email protected]>
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.

UI Bug - Deploying Stepper numbering
3 participants