-
Notifications
You must be signed in to change notification settings - Fork 227
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
Implemented UK_ENGLISH Locale #106
Conversation
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.
Glad to see UK English support!
Suggesting some small changes.
Codecov Report
@@ Coverage Diff @@
## master #106 +/- ##
==========================================
+ Coverage 35.68% 35.90% +0.22%
==========================================
Files 22 22
Lines 1390 1412 +22
==========================================
+ Hits 496 507 +11
- Misses 894 905 +11
Continue to review full report at Codecov.
|
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.
Almost there, Lets share the radius calculation as suggested, and remove the YAML discussing the previously-implemented 'remote' for city option. 👍
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.
Looks good!
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 tested the UK implementation on a Mac and it works as expected. Great job!
Implemented UK_ENGLISH Implemented Search
Description
Implemented UK_ENGLISH as an option for 'locale' in the same way USA_ENGLISH and CANADA_ENGLISH are implemented, except with 'co.uk' for Indeed and Monster.
This involved taking out the state or province in the URL for get_search_url in indeed.py and monster.py in subclass: IndeedScraperUKEng and MonsterScraperUKEng.
Context of change
Please add options that are relevant and mark any boxes that apply.
Type of change
Please mark any boxes that apply.
How Has This Been Tested?
Ran UK_ENGLISH in my_settings.yaml with cities in the UK such as London and Cambridge for Indeed and Monster.
Checklist:
Please mark any boxes that have been completed.