SC should not check Data Steam URL length

[damienen]

As it is encrypted it is often very long, but the unencrypted value is not so big. This change should probably be moved into the data marshal (it is also already implemented in FE). This causes issues such as these:

https://devnet-explorer.multiversx.com/transactions/3cbbecc15b949fa5887c80df9f7def55a428d71932109eb30ac3f7f5aab32aab#smart

The normal URL was around 150 chars, but the encrypted one is 320 so the mint fails.

[bucurdavid]

Fair enough for me

[bucurdavid]

Implemented #360

[damienen]

Cool. Closing.

[newbreedofgeek]

@damienen @bucurdavid SC change has removed the check for other URLs. This is wrong.

This should ONLY apply to Data Stream URL. reopening

[bucurdavid]

Changed only to Data Stream URL

[damienen]

@newbreedofgeek Please update SC

[damienen]

This has been solved. Closing

[fuzzyduke]

Issue with minting long URL seen again

https://devnet-explorer.multiversx.com/transactions/7f6785e45be85ee6fb616fea78eab218debe465128d569f125cd35544a3f1bc2

[bucurdavid]

This is due to title or description being too long.

require!(title.len() <= 30, ERR_TOO_MANY_CHARS);
require!(description.len() <= 400, ERR_TOO_MANY_CHARS);

Current implementation:

• Title: max 30 chars
• Description: max 400 chars

Maybe FE didn't catch the title length. @damienen

[Enciusan]

Yes here is a problem in FE, the validation was for 50 max chars for title. I will fix the validator there.

[damienen]

Solved. Closing.