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

Complete step once step is viewed #166987

Closed
wants to merge 1 commit into from
Closed

Conversation

bhavyaus
Copy link
Collaborator

@bhavyaus bhavyaus commented Nov 22, 2022

Fixes: #166747.
The checkbox now is toggled on select. The completion events now seem to be unnecessary?

Should we consider adding Next button as a part of this task itself or would we consider that after user feedback? cc: @daviddossett

Copy link
Member

@lramos15 lramos15 left a comment

Choose a reason for hiding this comment

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

Just find all references on the completion events before removing them to make sure they're no longer used.

@daviddossett
Copy link
Contributor

If we're doing this, should we also make the checkbox non-interactive? Not sure if there's still value in letting people manually toggle it.

Should we consider adding Next button as a part of this task itself or would we consider that after user feedback? cc: @daviddossett

Let's see how it feels first. I suspect we should add it but we can give it a try first.

@aeschli aeschli modified the milestones: November 2022, December 2022 Nov 30, 2022
@bhavyaus bhavyaus closed this Dec 28, 2022
@bhavyaus bhavyaus deleted the dev/bhavyau/getting-started branch December 29, 2022 18:28
@github-actions github-actions bot locked and limited conversation to collaborators Feb 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Complete walkthrough steps once viewed
5 participants