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

Make config for Kouka the same as JJ Dossen; add new "dev" config #28

Merged
merged 2 commits into from
Sep 2, 2020

Conversation

mogoodrich
Copy link
Member

No description provided.

@@ -1,38 +1,4 @@
{

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I moved all these out of "pih-config-liberia-harper-jjdossen" and into "pih-config-harper"... both JJ Dossen and Kouka use the "pih-config-harper" so they will now have the exact same components enabled.

I also removed "MCH" from the list so that it won't be enabled.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe @BranfordTGbieor wants MCH on Kouka for development?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @lnball, but I'm not sure we would need that for now. I suggested the dev of the forms reach a particular stage before we enable them on kouka for testings purposes. Maybe in the next two weeks or so...

@@ -6,39 +6,35 @@
"site": "HARPER",

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just the diff that results from replacing the "components" here with those defined in "pih-config-liberia-harper-jjdossen.json"

@@ -0,0 +1,39 @@
{
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new development configuration I set up that we can use on our development laptops. It has the exact same components as the "pih-config-liberia-harper", but also has MCH enabled so we can work on developing and testing that.

To use this, in your runtime properties you'd basically just set:
pih.config=liberia,liberia-harper,liberia-harper-dev

The components defined in "liberia-harper" would be overridden with the ones defined here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, that's good! That way we get the flexibility of not manually changing configs between environments. Is that correct @mogoodrich ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BranfordTGbieor it allows you to have a configuration setting you can use for development (of the MCH forms right now) which is different than what is on Kouka, so we can have Kouka show exactly what will go out to JJ Dossen.

@lnball
Copy link
Member

lnball commented Sep 1, 2020

Except for the one question about MCH on kouka, it looks good.

@mogoodrich
Copy link
Member Author

Thanks @lnball ... yeah, we were discussing that yesteday... my thought was prior to the release we make Kouka and JJ Dossen identical so we know exactly what is going out to JJ Dossen when testing.

Longer term, it might make sense to have both a staging server and a demo server (so in this case the staging server would have MCH turned on, but the demo server would not).

Copy link
Contributor

@BranfordTGbieor BranfordTGbieor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me @mogoodrich! Thanks.

@mogoodrich mogoodrich merged commit 08d4644 into master Sep 2, 2020
@mogoodrich
Copy link
Member Author

Merged this this, thanks Ellen and Branford.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants