Age | Commit message (Collapse) | Author |
|
This change was put under the wrong release as the result of a rebase
mistake.
|
|
Add install generator
|
|
|
|
Prior to this commit, users were required to manually configure this
extension after installing the gem. We add an install generator so users
can run a single command to handle this.
We include this command in the sandbox script so the sandbox is
ready to use this extension by default.
Co-authored-by: Noah Silvera <noah@super.gd>
|
|
#77 Add shipment shipped event
|
|
|
|
Transactions need to be reported to TaxJar when any of the shipment
ships (some stores have partial shipping). To accomodate this, we
add a new event which is fired whenever any shipment on an order ships.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
|
|
Update changelog for 0.18.2 release
|
|
|
|
Support order_recalculated event in Solidus < 2.11
|
|
|
|
Solidus < 2.11 doesn't support the `order_recalculated` event, so we
implement a decorator to support this feature in Solidus versions we
test against (2.10 and 2.9). We need this event to support sending order
updates to TaxJar, so we add it first.
Note that we include `SolidusSupport::EngineExtensions` so decorators
are loaded.
Co-authored-by: Nick Van Doorn <nick@super.gd>
Co-authored-by: Noah Silvera <noah@super.gd>
|
|
An upcoming commit will add a decorator that creates the
SuperGood::SolidusTaxjar::Spree namespace, which will change which module
all existing Spree instances in the lib folder reference.
We should force them to look for the root Spree namespace so tests
do not break.
|
|
Unlock the ExecJS version
|
|
|
|
A new release of ExecJS (2.8.1) patched a bug which was preventing the
app from booting, and autoprefixer, which indirectly requires
ExecJS for this app, temporarily locked ExecJS to a working version
until they fix a bug, so we no longer have to lock ExecJS ourselves.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
Relax Ruby required version to support Ruby 3.0+
|
|
|
|
|
|
Use spree address2 if it is present
|
|
|
|
The taxjar validation API has the option to take a freeform
street input consisting of the full address. This means that if both
address1 and address2 exist, we should concatenate them and pass them
to the street parameter.
This can help with situations that involve a suite number that has a
unique ZIP+4 code that can be determined from the suite. The suite
number is usually kept in address2, so without it, there's no way to
distinguish the full address if only the 5 digit zip code is passed.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
Add basic UI for Taxjar settings in solidus admin
|
|
Add a note in the changelog describing how this PR adds a admin
interface for basic taxjar settings.
|
|
To comply with the certification guidelines for Taxjar, we should
provide links to signup for Taxjar, and a link to a help article on
aquiring a Taxjar token.
|
|
This commit adds a tab to the taxes section of the solidus admin
settings. This provides the foundation for further configuration of
taxjar through the solidus admin.
The settings should be hidden if an API key has not been provided.
This is to conform to the certification guidelines for an official
Taxjar extension.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
Remove install generator instructions from readme
|
|
If you set the `ENV` variable in a test, this carries over to other
tests. This was affecting different tests which relied on the
environment.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
We are going to be writing feature specs for new admin user interface
components. Note that we can remove the line that requires
`rails_helper` as `feature_helper` also includes this.
Co-authored-by: Noah Silvera <noah@super.gd>
|
|
These instructions were added pre-emptively - currently no migrations
or install generators exist.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
Lock ExecJS and solidus extensions orb version
|
|
|
|
A new version of `solidusio/extensions` has been released which runs the
specs against Solidus 2.11. We are not ready for Solidus 2.11 yet, and
the failing CI is blocking things, so we will avoid Solidus 2.11 for
now, but we do want to support it eventually.
|
|
ExecJS released a minor version update with a breaking change. We want
to lock to the previous version (2.7) for now until a fix is released.
Co-authored-by: Nick Van Doorn <nick@super.gd>
Co-authored-by: Adam Mueller <adam@super.gd>
|
|
Add pry debugging tools
|
|
|
|
Pry stack explorer and byebug are useful tools to have for debugging
this extension.
|
|
Noah-Silvera/56-take-adjustments-into-account-when-calculating-shipping
Take adjustments into account when calculating shipping
|
|
|
|
Previously, the shipment_total method on order was used, which does
not take into account shipment promotions. Instead, we should manually
sum the shipment cost taking into account any adjustments.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
We are going to be changing the shipping calculator, so we should add a
simple spec to test the existing behavior.
Co-authored-by: Nick Van Doorn <nick@super.gd>
|
|
Add .circleci/config.yml
|
|
We want to switch from travis CI to circle CI as we have run out of
credits on travis, and circle CI has a robust orb for running tests
for solidus extensions.
|
|
Add TaxJar `nexus_regions` API call
|
|
Co-authored-by: Noah Silvera <noah@super.gd>
|
|
We want to start syncing nexus data from TaxJar to this extension. As
such we start by implementing an API call to fetch nexus regions from
TaxJar.
Future commits should offer path for users to persist this data in their
Solidus store.
Co-authored-by: Noah Silvera <noah@super.gd>
Co-authored-by: Adnan Abdulally <adnan.abdulally@printivity.com>
|
|
nvandoorn/update-pull-request-template-to-include-sandbox-test
Update PR template to include sandbox testing step
|
|
|
|
|