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

Make socials metadata optional #1108

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Conversation

JunichiSugiura
Copy link
Contributor

@JunichiSugiura JunichiSugiura commented Oct 26, 2023

Sozo fails to instantiate WorldMetadata when there's no social links defined on Scarb.toml thus uploading to ipfs fails.

.map(|v| v.try_into::<Metadata>().unwrap_or_default())

@JunichiSugiura JunichiSugiura added the bug Something isn't working label Oct 26, 2023
@JunichiSugiura JunichiSugiura changed the title Make socials metadata optional. Make socials metadata optional Oct 26, 2023
@JunichiSugiura JunichiSugiura force-pushed the fix(world)/metadata-optional-social branch 4 times, most recently from 7cd44bf to 997fbfc Compare October 26, 2023 17:02
@JunichiSugiura JunichiSugiura force-pushed the fix(world)/metadata-optional-social branch from 997fbfc to ee50171 Compare October 26, 2023 17:03
@tarrencev tarrencev merged commit 45fb6af into main Oct 26, 2023
11 checks passed
@tarrencev tarrencev deleted the fix(world)/metadata-optional-social branch October 26, 2023 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants