We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Passing an invalid property to update_metadata is accepted and breaks the chart. Specifically, I passed
update_metadata
dw.update_metadata(dw_id, properties = {'annotate' : 'SAMPLE ANNONATION'})
instead of:
dw.update_metadata(dw_id, properties = {'annotate' : {'notes' : 'SAMPLE ANNONATION'}})
The update was accepted and the corresponding property appeared in chart_properties.
chart_properties
As a result, the chart in the web UI became unusable. In addition, the "Refine" tab on the website vanished.
I suggest adding to the docs a list of valid properties to pass, and offering error messages when invalid ones are passed.
The text was updated successfully, but these errors were encountered:
Hello @robroc, thank you for your interest in our work!
If this is a bug report, please provide screenshots and minimum viable code to reproduce your issue, otherwise we can not help you.
Sorry, something went wrong.
thank you for this! and the suggestion!
Apologies for the delay but I'll be working on updating datawrapper in the following weeks! I'm hoping to address this.
datawrapper
chekos
No branches or pull requests
🐛 Bug Report
Passing an invalid property to
update_metadata
is accepted and breaks the chart. Specifically, I passeddw.update_metadata(dw_id, properties = {'annotate' : 'SAMPLE ANNONATION'})
instead of:
dw.update_metadata(dw_id, properties = {'annotate' : {'notes' : 'SAMPLE ANNONATION'}})
The update was accepted and the corresponding property appeared in
chart_properties
.As a result, the chart in the web UI became unusable. In addition, the "Refine" tab on the website vanished.
I suggest adding to the docs a list of valid properties to pass, and offering error messages when invalid ones are passed.
Environment
The text was updated successfully, but these errors were encountered: