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

feat(wizard_router): allow the user to complete the wizard flow #328

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

sappo
Copy link
Contributor

@sappo sappo commented Oct 16, 2023

No description provided.

@sappo sappo mentioned this pull request Oct 16, 2023
@sappo sappo changed the title [wizard_router] feat: allow the user to complete the wizard flow feat(wizard_router): allow the user to complete the wizard flow Oct 16, 2023
@sappo sappo closed this Oct 16, 2023
@sappo sappo reopened this Oct 16, 2023
@sappo sappo closed this Oct 16, 2023
@sappo sappo reopened this Oct 16, 2023
@spydon
Copy link
Contributor

spydon commented Feb 1, 2024

@sappo could you sign the CLA? Then we can merge this :)

@sappo
Copy link
Contributor Author

sappo commented Feb 1, 2024

Just did for the 2nd time, but the check is still failing

@spydon
Copy link
Contributor

spydon commented Feb 1, 2024

Just did for the 2nd time, but the check is still failing

Ah, it's due to that you don't have a default e-mail connected to the actions you take through the web interface I think, so your merges for example are connected with this e-mail: [email protected]

What you could do is just squash everything down to one commit that is up to date with the latest on main, or change the e-mail of the commits with the faulty e-mail to the e-mail that you signed the CLA with.

@spydon
Copy link
Contributor

spydon commented Feb 2, 2024

There, I fixed it for you @sappo :)

@spydon spydon enabled auto-merge February 2, 2024 08:30
@spydon spydon merged commit 2c88759 into canonical:main Feb 2, 2024
11 checks passed
@sappo
Copy link
Contributor Author

sappo commented Feb 2, 2024

@spydon thanks 👍

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.

2 participants