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 MangaDex #8

Merged
merged 1 commit into from
Apr 16, 2024
Merged

Conversation

hkamran80
Copy link
Member

@hkamran80 hkamran80 commented Apr 9, 2024

This PR removes the MangaDex logo. It is dependent on 2factorauth/twofactorauth#7979.

@hkamran80 hkamran80 added the update site Issue/PR updates information about a site in the repo. label Apr 9, 2024
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 Apr 16, 2024
@Carlgo11 Carlgo11 removed the inactive Inactive issue/PR, soon to be closed unless the author responds. label Apr 16, 2024
@hkamran80
Copy link
Member Author

2FA Directory (and the API) aren't showing the icon, causing the tests to fail.

@Carlgo11
Copy link
Member

Yeah I just tried debuggning it and it turns out the CF Cache is the reason it fails. I'll purge the cache.

@hkamran80
Copy link
Member Author

Ah. Thanks!

@Carlgo11 Carlgo11 marked this pull request as ready for review April 16, 2024 18:10
@Carlgo11 Carlgo11 merged commit c0825fa into 2factorauth:master Apr 16, 2024
2 checks passed
@hkamran80 hkamran80 deleted the update/mangadex branch April 16, 2024 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants