Support multiple accounts

Created on 14 October 2021, over 3 years ago
Updated 14 August 2023, over 1 year ago

At present Shipstation does not support shipping from Germany, however they will be supporting this in the future but we'd need to set up a separate account for this. We'd also need to use different Drupal Commerce Stores for this as the address would be different etc

It would be great to allow the module to be used for multiple accounts under one install, the other option would be for a sub module possibly?

✨ Feature request
Status

Needs review

Version

2.0

Component

Code

Created by

πŸ‡¬πŸ‡§United Kingdom c_archer Cumbria

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Merge request !8Update for multiple stores β†’ (Closed) created by c_archer
  • πŸ‡¬πŸ‡§United Kingdom c_archer Cumbria
  • πŸ‡ΊπŸ‡¦Ukraine khiminrm

    I've fixed the patch for the latest 2.x-dev.
    I haven't fully tested though.
    I would improve logic for the settings form. It's looks weird if we have only one store and there are a couple of the errors on first visit on the form after applying the patch. Maybe we can check some checkbox to enable per site support if there are multiple stores on a site?
    And I'm not sure why we need to generate API keys and use them there as the module actually doesn't make any requests to the ShipStation API (or I've not found if it does). We use custom username and password for the module's endpoint to verify requests from the ShipStation.

  • πŸ‡ΊπŸ‡¦Ukraine khiminrm

    I've created new branch and MR with some improvements in UI - admin form for the module.
    I've added a checkbox 'Use per store settings' and when it is checked - hide the user name and password fields and show grouped fields per each store. The checkbox is displayed when there are multiple stores on a site. We can add also other settings per each store if needed.

  • πŸ‡¬πŸ‡§United Kingdom c_archer Cumbria

    Roman this looks like a good start, I can see one blocker on the pr. If "Use per store settings" is checked its not clear to the user that they don't have to enter settings per each store. For example:

    User has:
    EU SS account
    UK SS account

    On the site they have these stores:
    EU Store
    UK Store
    US Store

    US store and UK store both use the UK SS account and the EU Store uses the EU SS Account. Does the user have the enter the UK SS account details twice?

    Final comment could we change "Use per store settings" label to "Shipstation Account per store" or something similar?

  • Status changed to Needs work 5 months ago
  • πŸ‡¬πŸ‡§United Kingdom c_archer Cumbria

    Roman just tried applying your MR against the latest version and it does not apply

  • Pipeline finished with Failed
    4 months ago
    Total: 143s
    #352112
  • Merge request !22Add filter for store β†’ (Open) created by c_archer
  • Pipeline finished with Failed
    4 months ago
    Total: 413s
    #353102
  • πŸ‡¬πŸ‡§United Kingdom c_archer Cumbria

    Created a new MR for this which simplifies the logic.

Production build 0.71.5 2024