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

[CMS-1254] Change blank country choice to "Choose a country" #187

Merged
merged 4 commits into from May 15, 2019

Conversation

kaedroho
Copy link
Contributor

@kaedroho kaedroho commented May 7, 2019

No description provided.

@codecov-io
Copy link

codecov-io commented May 7, 2019

Codecov Report

Merging #187 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #187   +/-   ##
=======================================
  Coverage   99.36%   99.36%           
=======================================
  Files          10       10           
  Lines         469      469           
=======================================
  Hits          466      466           
  Misses          3        3
Impacted Files Coverage Δ
directory_components/forms.py 96.66% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 423c554...1abea6e. Read the comment docs.

setup.py Outdated
@@ -3,7 +3,7 @@

setup(
name='directory_components',
version='10.8.3',
version='10.8.4',
Copy link
Contributor

@richtier richtier May 8, 2019

Choose a reason for hiding this comment

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

master is already at version 11 :)
https://github.com/uktrade/directory-components/blob/master/setup.py#L6

need to update the base branch

CHANGELOG.md Outdated

### Bug fixes:

- Changed default country selector text to "Select a country"
Copy link
Contributor

@sdonk sdonk May 8, 2019

Choose a reason for hiding this comment

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

Can you please add the Jira ticket number:
- CMS-1245

@sdonk sdonk force-pushed the 1254-change-blank-country-choice branch from e512df8 to 559f8a9 Compare May 15, 2019 09:43
@sdonk sdonk dismissed richtier’s stale review May 15, 2019 09:49

changelog updated

@sdonk sdonk merged commit 30ae1ee into master May 15, 2019
@sdonk sdonk deleted the 1254-change-blank-country-choice branch May 15, 2019 09:49
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

5 participants