-
-
Notifications
You must be signed in to change notification settings - Fork 182
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 social media schema field types #1967
Conversation
e1ea1a6
to
2234f3c
Compare
giving a second look I think we need to provide an upgrade step to cast those values to |
Do Twitter, Facebook and so on really only allow ASCII for the username? If you already check just discard my question, I'm only thinking aloud 😃 |
twitter_username, facebook_app_id and facebook_username should be ASCIILine instead of TextLine.
2234f3c
to
b8ab9ce
Compare
I need help with this; I don't understand Jenkins and I think failures are not related with my changes. |
There seems to be a general problem, see the first error on the console log: http://jenkins.plone.org/job/pull-request-5.1/1423/consoleFull |
found the issue, thanks! |
@plone/framework-team it's green, guys! |
twitter_username
,facebook_app_id
andfacebook_username
should beASCIILine
instead ofTextLine
.refs. plone/plone.app.upgrade#107
refs. plone/plone.app.layout#121