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

fixing template where we want to override nested attribute value #73

Merged

Conversation

nykolaslima
Copy link
Member

In immutable objects, when we try to override a nested attribute in the gimme call it doesn't work.

Now, after create the object with all property definitions, we take the properties that were not used in the constructor and starting processing then as deferred properties.

@ahirata @aparra can you guys review it please?

After merging it I'm going to release a fix version 2.2.1

@nykolaslima
Copy link
Member Author

Actually I'm going to release version 2.3.0 because we have new features, like uniqueRandom, cpf and cnpj functions.

@nykolaslima
Copy link
Member Author

Sorry guys, we'll need to release 3.0.0 because @aparra changed the functions' package and it will break the retrocompatibility

@ahirata
Copy link
Member

ahirata commented Jan 29, 2015

Looks good.

@nykolaslima
Copy link
Member Author

Thank you @ahirata :)

nykolaslima added a commit that referenced this pull request Jan 29, 2015
fixing template where we want to override nested attribute value
@nykolaslima nykolaslima merged commit 68d6c3b into six2six:master Jan 29, 2015
@nykolaslima nykolaslima deleted the override-nested-attribute-value branch January 29, 2015 00:35
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.

2 participants