Accept `fields` when POSTing to `/messages`

floraison / flack

a Rack app for the flor workflow engine

MIT License

9 stars 4 forks source link

Accept `fields` when POSTing to `/messages` #8

Closed jfrioux closed 7 years ago

jfrioux commented 7 years ago

Hi I implemented the support for fields, the same way I did for vars. I should have been more thorough the first time and deal with both in a single PR, sorry about that.

I reused the same tests, since it's really just a lookup in a hash, and I didn't want to add ~70 lines of code just for that.

Please let me know if that suits you.

Thanks

jmettraux commented 7 years ago

Hello Jeff,

no worries about multiple pull requests, I like to have many small commits that show the thought process.

Thanks a lot!