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

Remove smartphone #307

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

hobbitronics
Copy link
Contributor

IDP-24 Remove Smartphone check

@hobbitronics hobbitronics requested a review from a team as a code owner February 5, 2025 02:40
@hobbitronics hobbitronics requested review from briskt, forevermatt, mtompset and jason-jackson and removed request for a team February 5, 2025 02:40
Copy link

sonarqubecloud bot commented Feb 5, 2025

Copy link
Contributor

@briskt briskt left a comment

Choose a reason for hiding this comment

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

Thanks. Looks like a couple PRs were combined.

@hobbitronics hobbitronics changed the base branch from main to improve-mfa-card-layout February 5, 2025 23:21
@hobbitronics hobbitronics merged commit 053a2b1 into improve-mfa-card-layout Feb 5, 2025
3 checks passed
@hobbitronics hobbitronics deleted the remove-smartphone branch February 5, 2025 23:21
@briskt
Copy link
Contributor

briskt commented Feb 6, 2025

I believe it works better to merge the "parent" first. That is, the one that's merging into the main branch. In that way, each PR's description stays accurate to the commits within it.

@hobbitronics
Copy link
Contributor Author

hobbitronics commented Feb 6, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants