-
Notifications
You must be signed in to change notification settings - Fork 429
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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 |
Introduced a check on the 'settings' object (in Community Settings) so that it's not an empty object. If no 'avatar_url' or 'cover_url' is set, the object will now reflect this.
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
![grafik](https://private-user-images.githubusercontent.com/111915936/399068685-2548a875-265c-4bde-9d16-c4135b1fd177.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzODU3NjEsIm5iZiI6MTczOTM4NTQ2MSwicGF0aCI6Ii8xMTE5MTU5MzYvMzk5MDY4Njg1LTI1NDhhODc1LTI2NWMtNGJkZS05ZDE2LWM0MTM1YjFmZDE3Ny5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMlQxODM3NDFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hOGQyNGQ0NjRhOTZmYTU2MGY0ZGZhMDU5ZDVhMmEyNGExNDQ0YWI4YjlhZjMyOWM1NjUxZTAzN2E1OGNmNzgyJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.lHpDcW4vVGeGCk8BQ-hOcqndpyatsy9i6aspLzY5-B4)
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: