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

Increase Workshop Publication Requirements #4424

Conversation

TechnicalitiesDevelopment
Copy link
Contributor

@TechnicalitiesDevelopment TechnicalitiesDevelopment commented Oct 19, 2024

What is the new feature?

It increases the checks made before publishing on the workshop to make it impossible to publish mods without changing their icon or description. It also checks for a description with fewer than 50 letters/digits to publish and scales up the icon.png file when publishing. If the workshop icon is from another folder it will be copied over to the mod sources folder as well.

Why should this be part of tModLoader?

See #3643

Are there alternative designs?

Heavy moderation of the workshop

Porting Notes

If they have a mod with the default description or icon then they have to change that prior to updating.

@Mirsario Mirsario self-assigned this Oct 19, 2024
Copy link
Member

@Mirsario Mirsario left a comment

Choose a reason for hiding this comment

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

  • icon_workshop.png is no longer required but will be automatically created from the chosen preview image, upscaling it to at least 480x480 if needed. Users are notified of this in GUI:
    image
  • Descriptions must be present and have at least not 20 but 50 characters, counting only letters and digits.

@TechnicalitiesDevelopment
Copy link
Contributor Author

Thank you, I will edit the description

@Mirsario Mirsario requested a review from Solxanich October 21, 2024 13:11
@JavidPack JavidPack merged commit 8f9dfca into tModLoader:1.4.4 Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants