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

Fix text confirming investment heading support #3656

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Conversation

javierm
Copy link
Member

@javierm javierm commented Jul 8, 2019

Background

We accidentally removed the count option in the confirm group text in commit 55fb14a, which made the translation return a hash.

Visual Changes

Before this change:

Confirmation text shows Object object

After this change:

Confirmation text shows a useful message

@javierm javierm self-assigned this Jul 8, 2019
@javierm
Copy link
Member Author

javierm commented Jul 8, 2019

Travis build failed because of the issue fixed by #3641, so the failure is not related to this pull request.

@javierm javierm force-pushed the fix_alert_text branch 3 times, most recently from 18f7b42 to efd3a76 Compare August 9, 2019 12:11
We accidentally removed the `count` option in commit 55fb14a, which
made the translation return a hash.

The test is a bit hacky, which makes me think changing the user
interface would probably be a better solution.
@javierm javierm added this to Reviewing in Roadmap Sep 11, 2019
@javierm javierm added the Bug label Sep 11, 2019
@javierm javierm merged commit 88afeda into master Sep 11, 2019
Roadmap automation moved this from Reviewing to Release 1.1.0 Sep 11, 2019
@javierm javierm deleted the fix_alert_text branch September 11, 2019 13:34
smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
Fix text confirming investment heading support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Roadmap
  
Release 1.1.0
Development

Successfully merging this pull request may close these issues.

None yet

1 participant