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 #15977 - change enc output in a compatible way #3703

Closed
wants to merge 1 commit into from

Conversation

timogoebel
Copy link
Member

No description provided.

@dLobatog
Copy link
Member

dLobatog commented Aug 5, 2016

Fair enough, although this will be a breaking change for 1.12 users who rely on it 😄
Can you call it organization_title? It's just because we use _title in some places in the application to refer to the full name. Thanks @timogoebel !

@timogoebel
Copy link
Member Author

Fair enough, although this will be a breaking change for 1.12 users who rely on it 😄

Yes, I know. Too bad I didn't spot this earlier. Naming ENC parameters is always a hard thing. Will change the parameter name in a sec.

@dLobatog
Copy link
Member

dLobatog commented Aug 5, 2016

Merged as b96e021, thanks @timogoebel!

@dLobatog dLobatog closed this Aug 5, 2016
@timogoebel
Copy link
Member Author

@domcleal / @dLobatog : Can we put this on 1.12.2? What do you think? I'd be in favor of it.

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.

3 participants