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

Added canPrePopulate() method. Fixes #87 #88

Merged
merged 1 commit into from
Oct 3, 2017
Merged

Added canPrePopulate() method. Fixes #87 #88

merged 1 commit into from
Oct 3, 2017

Conversation

pointybeard
Copy link
Member

Relates to #87 and symphonycms/symphonycms#2751

Prior to this change in 2.7.x, the Publish: New page never actually checked to see if a field was allowed to prepopulate. The SBL field was missing the canPrePopulate method, so meant prepopulating stopped working.

This PR added that method in, bringing SBL field up to date with new behaviour in 2.7.x.

@pointybeard
Copy link
Member Author

Not entirely sure who is maintaining this repo. Are you happy for me to go ahead and merge?

@jensscherbl
Copy link
Member

Everything is maintained by @nitriques these days. Not sure about merging yourself as I don’t know about Nicolas’ release workflow.

@brendo brendo merged commit 2180364 into symphonycms:master Oct 3, 2017
@nitriques
Copy link
Member

nitriques commented Oct 6, 2017

@pointybeard Hi! Thanks for the PR!

@brendo yeah, @jensscherbl's is right. Please request a review instead!

@nitriques
Copy link
Member

It's been released as 2.0.2

@pointybeard
Copy link
Member Author

No problems! Thanks for the speedy release.

@nitriques
Copy link
Member

👍

@nitriques nitriques self-assigned this Oct 10, 2017
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.

4 participants