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

The array from the JSON config file is passed as an object #901

Open
mercs600 opened this issue Jul 11, 2023 · 1 comment
Open

The array from the JSON config file is passed as an object #901

mercs600 opened this issue Jul 11, 2023 · 1 comment
Labels
read Read by sequences

Comments

@mercs600
Copy link

Describe the bug
I have a JSON config file similar to this one:

{
        "sequences": [
            "products_from_service,
            "products_mapper",
            "products_to_service"
        ]

}

On the first run, I use si seq deploy mysequence -f config.json and then I get the correct Array in the application.
But when mysequence is run and it tries to pass the current config to another sequence by using:
sequenceClient.start({ appConfig: this.config }) then my Array is then converted to JSON:

    sequences: {
      '0': 'products_from_akeneo',
      '1': 'products_mapper',
      '2': 'products_to_baselinker'
    }

Steps to Reproduce

  1. Run your sequence with the config file (which includes an Array) - si seq deploy mysequence -f config.json
  2. From your "mysequence", run another sequence and pass the current config there sequenceClient.start({ appConfig: this.config })

Expected behavior
There should be consistent types?

Version (please complete the following information):

  service: '@scramjet/host',
  apiVersion: 'v1',
  version: '0.34.4',
  build: '344459e'

Additional information like logs, screenshots etc.

@MichalCz
Copy link
Member

We're look into this. I'll update if we have more info.

@scramjet-bot scramjet-bot added read Read by sequences and removed read Read by sequences labels Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
read Read by sequences
Projects
None yet
Development

No branches or pull requests

3 participants