Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow users/merchants to enter a sandbox API token for testing #36

Open
Noah-Silvera opened this issue Feb 3, 2021 · 3 comments
Open
Labels
enhancement New feature or request needs-certification-acceptance This ticket is done pending it's acceptance in the certification process

Comments

@Noah-Silvera
Copy link
Member

Currently the Taxjar api token is passed through an environment variable. We also need to allow consumers of this extension to enter their Taxjar sandbox API token in an easy way.

This could be accomplished through an UI on the solidus admin. Currently, no UI exists for the extension on the solidus admin side, so this ticket is a good candidate for also creating that basic admin interface.

@Noah-Silvera Noah-Silvera added this to the TaxJar Certification milestone Feb 3, 2021
@Noah-Silvera Noah-Silvera added the enhancement New feature or request label Feb 3, 2021
@jarednorman
Copy link
Member

#32 begins adding some UI.

@jarednorman
Copy link
Member

Solidus provides a preferences system that might be of help here. Typically we prefer to statically define things, but for situations like this it can be convenient to give users the ability store these things in the UI. Would probably want to rely on encrypted preferences if available.

@Noah-Silvera Noah-Silvera removed this from the TaxJar Certification milestone Nov 19, 2021
@Noah-Silvera
Copy link
Member Author

Pulled this out of the certification milestone as it is an optional requirement. It's also possible to set via an environment variable at the moment, so we may actually meet the requirement regardless. We shall see when we enter the first round of certification and potentially revisit it down the line.

@Noah-Silvera Noah-Silvera added this to the TaxJar Certification milestone Jan 3, 2022
@Noah-Silvera Noah-Silvera added the needs-certification-acceptance This ticket is done pending it's acceptance in the certification process label Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-certification-acceptance This ticket is done pending it's acceptance in the certification process
Projects
None yet
Development

No branches or pull requests

2 participants