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

Update Kenya static site branch with configurations #12

Open
wants to merge 22 commits into
base: kenya
Choose a base branch
from

Conversation

wcyn
Copy link

@wcyn wcyn commented Jul 11, 2017

What does this PR do?

It adds the generic configuration settings to the kenya branch to reduce hard coding of values

Changes Made

  • Pulled in changes from the generic branch develop
  • Added configurations for Kenya jurisdiction

@wcyn wcyn requested a review from DavidLemayian July 11, 2017 09:38
s3_website.yml Outdated
@@ -1,6 +1,6 @@
s3_id: <%= ENV['AWS_ACCESS_KEY_ID'] %>
s3_secret: <%= ENV['AWS_SECRET_ACCESS_KEY'] %>
s3_bucket: opengazettes.or.ke
s3_bucket: nigeria.opengazettes.co

Choose a reason for hiding this comment

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

Probably should be Kenya.

Copy link
Author

Choose a reason for hiding this comment

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

@DavidLemayian Thanks. That's the config file I've been failing to change. I've moved the setting to use the environment variable set from the .env file

@@ -8,7 +8,12 @@
# Site settings
title: Open Gazettes Kenya
email: [email protected]
url: http:https://opengazettes.or.ke
url: http:https://opengazettes.or.ng

Choose a reason for hiding this comment

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

Probably should be Kenya.

@DavidLemayian
Copy link

Hey @andela-wcyn. ☝️

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

2 participants