instedd / mbuilder

Scalable SMS application builder
http://mbuilder.instedd.org/
6 stars 1 forks source link

Update/create Resourcemap facility from Mbuilder is not working #220

Closed mmuller closed 9 years ago

mmuller commented 9 years ago

Found in Version: 1.1.1

nditada commented 9 years ago

Does the current Resmap hub connector support R/W? See #245

nditada commented 9 years ago

This issue is pre-hub, we might be able to just close it.

bcardiff commented 9 years ago

mbuilder-resmap does not go through hub. Yet i would expect to be fixed already this issues.

— Sent from Mailbox

On Fri, Mar 6, 2015 at 5:00 PM, nditada notifications@github.com wrote:

This issue is pre-hub, we might be able to just close it.

Reply to this email directly or view it on GitHub: https://github.com/instedd/mbuilder/issues/220#issuecomment-77625387

mmuller commented 9 years ago

Tested in version 1.2-pre2

Create Facilities is working. Facilities cannot be updated from mBuilder. Only the GET request is sent to Resourcemap (POST is not sent).

suggestion by @bcardiff:

https://github.com/instedd/mbuilder/blob/1.2-pre2/app/models/tables/resource_map.rb#L48 should call to https://github.com/instedd/mbuilder/blob/1.2-pre2/app/models/database_execution_context.rb#L74

pmallol commented 9 years ago

Ok in version 1.2-pre3