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

Update Moxfield API integration according to new authentication method #273

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

ndepaola
Copy link
Collaborator

Description

Checklist

  • I have installed pre-commit and installed the hooks with pre-commit install before creating any commits.
  • I have updated any related tests for code I modified or added new tests where appropriate.
  • I have manually tested my changes as follows:
    • Checked the Moxfield integration is turned off if no secret is set
    • Checked Moxfield decklists can be imported if the secret is valid
  • I have updated any relevant documentation or created new documentation where appropriate.
    • Will do this separately after merging - github wikis are managed as their own git repos

@ndepaola ndepaola added the bug Something isn't working label Dec 29, 2024
@ndepaola ndepaola self-assigned this Dec 29, 2024
@ndepaola ndepaola merged commit 98198f8 into master Dec 29, 2024
3 checks passed
@ndepaola ndepaola deleted the moxfield-api-authentication branch December 29, 2024 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

Moxfield links no longer importing
1 participant