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 wearables and skins max size #3049

Open
7 of 10 tasks
braianj opened this issue Feb 29, 2024 · 1 comment
Open
7 of 10 tasks

Update wearables and skins max size #3049

braianj opened this issue Feb 29, 2024 · 1 comment
Labels
enhancement A feature or a fix that has low priority

Comments

@braianj
Copy link
Contributor

braianj commented Feb 29, 2024

Description

The current upload size limit for the Skin Category is set at 2MB, and the wearable size (others than skins) is restricted to 2MB as well. We need to apply this proposal, to upload size for the Skin Category from 2MB to 8MB.

Update Validation Rules and Constants for Item Sizes

  • Define the following:

    • The maximum size for wearables is 2 MB.
    • The maximum size for a skin is 8 MB.
    • The maximum size for a thumbnail is 1 MB.
    • The maximum size for emotes is 2 MB.
    • The combined size of a wearable plus a thumbnail is 3 MB.
    • The size for a skin plus a thumbnail is 9 MB.
  • Update Validation in content-validator and its constants (PR)

  • Add Necessary Constants in builder-client in the file src/files/constants.ts

  • Update Validation Processes in Builder (Current PR):

Create ADR for Size Updates:

  • Create an ADR documenting the size updates, referring to ADR-51 and ADR-45 for this update.

Link

@braianj braianj added the enhancement A feature or a fix that has low priority label Feb 29, 2024
@nachomazzara
Copy link
Contributor

Should we need to review the UI/UX for displaying size errors?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A feature or a fix that has low priority
Projects
None yet
Development

No branches or pull requests

2 participants