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

Custom Attributes example #3

Closed
vlatkoB opened this issue Oct 24, 2018 · 5 comments
Closed

Custom Attributes example #3

vlatkoB opened this issue Oct 24, 2018 · 5 comments
Labels
docs Add or fix documentation or examples

Comments

@vlatkoB
Copy link

vlatkoB commented Oct 24, 2018

Hi, just testing this nice app, but can't figure what format should be used for Custom Attributes in web interface. Always getting Parse error, check your formatting and try again.
Tried x, #x, #x=y, but nothing works

@aviaviavi
Copy link
Owner

Ah sorry for the confusion! Custom attributes are

key=val,key2=val2

I'll make sure this is documented better

@aviaviavi aviaviavi added the docs Add or fix documentation or examples label Oct 24, 2018
@aviaviavi
Copy link
Owner

Oops wait, you've uncovered an issue!

The toodles server looks for key=val, but the UI demands key:val. I'll fix :)

@vlatkoB
Copy link
Author

vlatkoB commented Oct 24, 2018

I'd also like to mention that web UI on edit action doesn't populate fields with current values.

@aviaviavi
Copy link
Owner

aviaviavi commented Oct 24, 2018

Right, this was a choice I made more because currently tags and custom attributes are add-only via the UI. It wasn't clear what the right way to have the UI handle edits that would be applied to multiple todo's at once.

I think that's an issue that can be fixed when #5 is addressed

@aviaviavi
Copy link
Owner

Ok the UI properly expects =, so x=y works via the UI now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Add or fix documentation or examples
Projects
None yet
Development

No branches or pull requests

2 participants