You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Perviously, that helper was making minor changes to the context passed from our place-detail view into our place-detail template. But with our new dynamic forms feature, we need to make a lot more changes to the context as we pass it into our template. Thus, we added a lot more pre-processing logic via #435, and the each_place_item helper method has become quite long. We should try and move that method into our view, and be explicit that it is performing a pre-processing task.
The text was updated successfully, but these errors were encountered:
In PR #435 we added lots of extra logic into our
each_place_item
Handlebars Helper, shown here: https://github.com/smartercleanup/platform/pull/435/files#diff-84b7065ac0f6c9065befa0b524ce0d81L174Perviously, that helper was making minor changes to the context passed from our
place-detail
view into ourplace-detail
template. But with our new dynamic forms feature, we need to make a lot more changes to the context as we pass it into our template. Thus, we added a lot more pre-processing logic via #435, and theeach_place_item
helper method has become quite long. We should try and move that method into our view, and be explicit that it is performing a pre-processing task.The text was updated successfully, but these errors were encountered: