-
Notifications
You must be signed in to change notification settings - Fork 430
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
UpdateProps of community settings is rejected by Hivemind as invalid #3932
Comments
I've been working on a fix for this and in doing so, I broke Hivemind. So ety001 fixed Hivemind. Which appears to have fixed the issue with community images updating. Would you mind checking and confirming this for me please? |
@the-gorilla-steem I would check this, but where can I see your changes? |
@only-dev-time - the Hivemind update has already been put live so it's available on steemit.com. It appears that I don't need to update condenser. |
@the-gorilla-steem I tested it with DU. The error still exists (I removed the old cover url before):
The fix for Hivemind concerns The still existing problem is that the field |
Updating the community props with
custom_json
op contains the fieldsettings
which is an empty object if no image has been set.Example
Mention by user: https://steemit.com/@event-horizon/sp64bx
Transaction: https://steemworld.org/block/91602709/91602709-13
Hivemind
Hivemind doesn't accept empty objects in the fields:
link to the check: https://github.com/only-dev-time/hivemind/blob/33204e1c2a18551acdb60e0a6142d1e2a4983d71/hive/indexer/community.py#L89
Analysis
With #3927 in
SettingsEditButton.jsx
the fieldsettings
was added to the props for thecustom_json
:only-dev-time@0da5aa3#diff-0b3b832a35933fe6e6ab1230a31131d076a671c39806fc56863578bcdcb39e83R100
@the-gorilla-steem Could a check that no images has been set solve the problem?
The text was updated successfully, but these errors were encountered: