search

adlio / trello

Trello API wrapper for Go
MIT License
220 stars 71 forks source link

Add basic support for CustomFields #21

Closed mangelajo closed 5 years ago

coveralls commented 6 years ago

Coverage Status

Coverage increased (+1.4%) to 71.973% when pulling 7befe39d5dcee977ea9f44604aa49680869173f4 on mangelajo:customFields into 05dcd358e32866f2353c4f49077346a0eb585436 on adlio:master.

mangelajo commented 6 years ago

Let me know if it makes sense as it is, or if we want something changed. I'm using it as it is.

adlio commented 6 years ago

Just got back from vacation. Hoping to review this week. Intending to merge after review.

mangelajo commented 6 years ago

thank you, let me know if you believe that some of the interfaces could be improved. For the custom fields to be usable I needed to create some helpers that process the raw trello data into something more digestable.

El mar., 3 jul. 2018 9:05 p. m., Aaron Longwell notifications@github.com escribió:

Just got back from vacation. Hoping to review this week. Intending to merge after review.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/adlio/trello/pull/21#issuecomment-402262160, or mute the thread https://github.com/notifications/unsubscribe-auth/ABHzckxv3VV1OmNIR8L8xNNojRPEcpf8ks5uC8CKgaJpZM4Uxyhf .

mangelajo commented 5 years ago

Thanks @adlio , I had another look to the code in hope of finding a better interface, but I couldn't come up with something better.

I'm using it from here so far: https://github.com/mangelajo/track/blob/master/cmd/trelloListCards.go#L84

gesellix commented 5 years ago

I would like to see this merged :)

adlio commented 5 years ago

@gesellix Ask and ye shall receive!

gesellix commented 5 years ago

@adlio oh, if life would always be so easy :-p Thanks so much for the quick merge!

mangelajo commented 5 years ago

Thank you! :)

irc: ajo / mangelajo Miguel Angel Ajo Pelayo +34 636 52 25 69 skype: ajoajoajo

On Tue, Oct 9, 2018 at 11:20 PM Tobias Gesellchen notifications@github.com wrote:

@adlio https://github.com/adlio oh, if life would always be so easy :-p Thanks so much for the quick merge!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/adlio/trello/pull/21#issuecomment-428356738, or mute the thread https://github.com/notifications/unsubscribe-auth/ABHzciotYH8HEBhDwktS8mMG5VWKvN8lks5ujRMmgaJpZM4Uxyhf .