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

Single passkey only on eBay and PayPal #43

Merged
merged 4 commits into from
Mar 19, 2025

Conversation

arxo
Copy link
Contributor

@arxo arxo commented Mar 8, 2025

solves #16 and #17

arxo added 3 commits March 8, 2025 11:47
checks complained about regions:

Error: -cu is not a valid region code
Error: -kp is not a valid region code
Error: -ir is not a valid region code
Error: -sy is not a valid region code
Error: -ve is not a valid region code
Error: -ru is not a valid region code
Error: -tr is not a valid region code

Signed-off-by: arxo <[email protected]>
@arxo
Copy link
Contributor Author

arxo commented Mar 8, 2025

someone should have a look at the PR validator failure...

Copy link

As the requested changes haven't been resolved yet, this PR will be marked as inactive.

Warning
Unless there's activity within the next 7 days, this PR will be closed.

@github-actions github-actions bot added the inactive Inactive issue/PR, soon to be closed unless the author responds. label Mar 16, 2025
@hkamran80 hkamran80 added multiple sites Issue/PR includes multiple sites. update site Issue/PR updates information about a site in the repo. and removed inactive Inactive issue/PR, soon to be closed unless the author responds. labels Mar 16, 2025
@arxo
Copy link
Contributor Author

arxo commented Mar 16, 2025

Copy link
Member

@hkamran80 hkamran80 left a comment

Choose a reason for hiding this comment

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

Apologies for the delay in reviewing your PR.
You can ignore the external tests error.

Comment on lines 31 to 39
"regions": [
"-cu",
"-kp",
"-ir",
"-sy",
"-ve",
"-ru",
"-tr"
]
Copy link
Member

Choose a reason for hiding this comment

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

Why did you remove the regions array?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because the JavaScript tests complained about every single one of the regions until I removed all of them.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll revert that particular change to see if it somehow works now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

JS test now fails again.

hopefully the failing JS tests were just temporary

Signed-off-by: arxo <[email protected]>
@Carlgo11 Carlgo11 merged commit bba08b7 into 2factorauth:master Mar 19, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multiple sites Issue/PR includes multiple sites. update site Issue/PR updates information about a site in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants