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

Integrate *Activation* process into scheduled runs. #83

Open
mathomas opened this issue Jul 5, 2018 · 0 comments
Open

Integrate *Activation* process into scheduled runs. #83

mathomas opened this issue Jul 5, 2018 · 0 comments
Assignees
Projects

Comments

@mathomas
Copy link
Contributor

mathomas commented Jul 5, 2018

Possible Options (we're leaning strongly toward the first):

  • Have activation just run as another step in the main index.js of the existing Lambda , along with introduction (same function, more functionality)
    • pro: simplest approach, easy code reuse (just refactor existing code to fit)
    • con:
      • must always process both introduction and activation at same time
      • can't be scheduled separately, as currently written/conceived, failure of introduction will cause activation not to happen (probably not a huge problem)
      • possibility of Lambda timeout (but doubtful?).
  • Use URL parameter/path to separate the functionality, but on the same endpoint
    • pro:
      • introduction/activation can be scheduled separately and have different result objects
      • easy code reuse
    • con: if/else to split functionality in index.js-- lack of cohesion
  • Brand-new function endpoint/module in same project
    • pro: separation of concerns
    • con: difficulty of code reuse
  • Separate project
    • pro: separation of concerns
    • con: complexity:
      • separate CodeStar projects
      • separate Github repos
      • difficulty of code reuse
@mathomas mathomas created this issue from a note in slink-board (To do) Jul 5, 2018
@mathomas mathomas changed the title Integrate *Activation* function into scheduled runs. Integrate *Activation* process into scheduled runs. Jul 6, 2018
@krzysztofwasiak-wipro krzysztofwasiak-wipro self-assigned this Jul 10, 2018
@krzysztofwasiak-wipro krzysztofwasiak-wipro moved this from To do to In progress in slink-board Jul 10, 2018
kwasiak pushed a commit that referenced this issue Jul 10, 2018
@krzysztofwasiak-wipro krzysztofwasiak-wipro moved this from In progress to Done in slink-board Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
slink-board
  
Done
Development

No branches or pull requests

2 participants