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(AnimeDrive): add support for iFrame and Hungarian description #8343

Merged
merged 6 commits into from
May 3, 2024

Conversation

ferivoq
Copy link
Contributor

@ferivoq ferivoq commented May 1, 2024

Description

I added support so know you can see how much is left from an Anime.
Added hungarian translation to the metadata.json and the iframe things.

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected

animedrive-2
animedrive-1

@ferivoq
Copy link
Contributor Author

ferivoq commented May 1, 2024

Could somebody check what's wrong??

@yamiOnEstrogen
Copy link
Contributor

Hello @ferivoq! Please make sure your version number has been bumped up!

Version: 1.0.1 > 1.0.2

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

ferivoq commented May 1, 2024

Could somebody merge pls if it's good?

@yamiOnEstrogen
Copy link
Contributor

@theusaf

@theusaf theusaf changed the title chore(AnimeDrive): Added support for iFrame + Hungarian description feat(AnimeDrive): add support for iFrame and Hungarian description May 1, 2024
websites/A/AnimeDrive/metadata.json Show resolved Hide resolved
"logo": "https://cdn.rcd.gg/PreMiD/websites/A/AnimeDrive/assets/logo.png",
"thumbnail": "https://cdn.rcd.gg/PreMiD/websites/A/AnimeDrive/assets/thumbnail.png",
"version": "1.0.2",
"logo": "https://cdn.discordapp.com/icons/702098578251841577/be50b5dbcb30bd99512b6df29284db0c.webp",
Copy link
Member

Choose a reason for hiding this comment

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

Logos should be 512x512px. You may need to upload the large image to another image source to avoid issues with query parameters with the current schema.

Suggested change
"logo": "https://cdn.discordapp.com/icons/702098578251841577/be50b5dbcb30bd99512b6df29284db0c.webp",
"logo": "https://i.imgur.com/VlBtvJo.png",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@ferivoq
Copy link
Contributor Author

ferivoq commented May 2, 2024

If it's good can you merge? @theusaf

@theusaf theusaf enabled auto-merge (squash) May 2, 2024 17:49
@theusaf theusaf requested a review from Bas950 May 2, 2024 17:49
@theusaf theusaf merged commit 8e92a1f into PreMiD:main May 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants