Skip to content

[WIP] Feature multi apps#43

Open
edgurgel wants to merge 15 commits intomasterfrom
feature-multi-apps
Open

[WIP] Feature multi apps#43
edgurgel wants to merge 15 commits intomasterfrom
feature-multi-apps

Conversation

@edgurgel
Copy link
Copy Markdown
Owner

@edgurgel edgurgel commented Sep 6, 2015

It closes #27

This PR will make Poxa accept a new parameter as part of the configuration:

apps: [{"app_id2", "app_key2", "secret2"}]

The apps parameter will need to map nicely to something that conform understands so poxa.conf will be simple to maintain. The above configuration may change.

The old way will also be accepted and merged with this new parameter.

Missing:

  • Change app configuration check from poxa.ex so it outputs how many apps are configured.
  • More integration tests to test isolation between apps;
  • Update poxa.conf to support multiple apps;
  • Update README with the new way to configure multiple apps.

@arrowcircle
Copy link
Copy Markdown

@edgurgel How its going with multiapps? I have thoughts about configuration.
First of all, if key and secret set via env, it should run in single app mode ignoring config.
Multiapp is only usable via config file. What do you think?

@edgurgel
Copy link
Copy Markdown
Owner Author

edgurgel commented Nov 9, 2015

I would like to use just the config file through conform, but I can't find a nice way to handle a list of configured apps through conform's DSL :/

@Alir3z4
Copy link
Copy Markdown

Alir3z4 commented Jul 14, 2020

Wish this could get any progress.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for multiple apps via config

3 participants