-
Notifications
You must be signed in to change notification settings - Fork 10
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
[WEBSITE-899] Removed opensearch builder and request to separate gem. #61
[WEBSITE-899] Removed opensearch builder and request to separate gem. #61
Conversation
lib/parliament/request.rb
Outdated
# @attr_reader [String] base_url the base url of our api. (expected: http://example.com - without the trailing slash). | ||
# @attr_reader [Hash] headers the headers being sent in the request. | ||
# Namespace for classes and modules that handle http connections. | ||
# @since 0.7.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incorrect indentation detected (column 0 instead of 2).
e171253
to
d6846a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've created a JIRA ticket for my below comment: WEBSITE-952 which we should pull into next sprint
@@ -170,8 +170,8 @@ def house_membership_status | |||
def build_house_membership_status(no_current_seat_incumbency, no_current_house_incumbency, former_lord, former_mp) | |||
statuses = [] | |||
statuses << 'Current MP' unless no_current_seat_incumbency | |||
statuses << 'Lord' unless no_current_house_incumbency | |||
statuses << 'Former Lord' if former_lord | |||
statuses << 'Member of the House of Lords' unless no_current_house_incumbency |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to look at extracting these into a language file going forward
No description provided.