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

Fixes #709 : Remove memoization to avoid stale localized state name #716

Merged
merged 5 commits into from Oct 15, 2020

Conversation

the-spectator
Copy link
Contributor

Fixes #709

Remove memoization to avoid stale localized state name

Remove memoization to avoid stale localized state name
@anilmaurya
Copy link
Member

Hi @the-spectator

A spec verifying bug fix will be good to have.

@codeclimate
Copy link

codeclimate bot commented Oct 13, 2020

Code Climate has analyzed commit 6fcd323 and detected 0 issues on this pull request.

View more on Code Climate.

@the-spectator the-spectator marked this pull request as ready for review October 13, 2020 20:15
@the-spectator
Copy link
Contributor Author

@anilmaurya Added specs. Please have a look at it.

@anilmaurya anilmaurya merged commit 8560d40 into aasm:master Oct 15, 2020
@anilmaurya
Copy link
Member

Great @the-spectator , thank you for contributing 🏅

@the-spectator the-spectator deleted the display_name_709 branch October 15, 2020 09:32
@the-spectator
Copy link
Contributor Author

@anilmaurya will it be alright if this PR to get labeled as hacktoberfest-accepted ?

@anilmaurya
Copy link
Member

Done @the-spectator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

human_state cached across locales in 5.1.1 release
2 participants