> Why couldn't currency and tax_total just be returned by default Spree?

Sounds good to me!  PR: https://github.com/spree/spree/pull/4462

A couple things though:

1. I don't think "tax_total" here is doing anything anymore -- it [seems to have been renamed](https://github.com/spree/spree/blob/b0d5242/core/db/migrate/20131118050234_rename_tax_total_fields.rb#L5) recently and the new columns [seem to already be included](https://github.com/spree/spree/blob/10ad76c/api/app/helpers/spree/api/api_helpers.rb#L77) in default Spree.  Let me know if I'm reading that wrong.

2. The other parameters (:visit_id, :visitor_id, :pageview_id, :last_pageview_id) seem to be [from spree_jirafe](https://github.com/spree/spree_jirafe/blob/7730cb3/db/migrate/20131204192804_add_visit_info_to_spree_orders.rb#L3-L6).  Is it strange that they are a part of spree_hub_connector or is that intended?  e.g, I don't think I can test them easily in this extension (see @radar's [comment](https://github.com/spree/spree_hub_connector/pull/20#discussion_r10730432)) because the columns don't exist in default Spree.