- Issue created by @solideogloria
- First commit to issue fork.
- πΊπΈUnited States jrockowitz Brooklyn, NY
jrockowitz β changed the visibility of the branch 3462738-sid-schema to hidden.
- Merge request !692Issue #3462738: Make column webform_submission_data.sid signed β (Open) created by jrockowitz
- πΊπΈUnited States jrockowitz Brooklyn, NY
Good catch!
My only concern is how the update hook handles changing the 'webform_submission_data.sid' schema when there are millions of records.
- πΊπΈUnited States jrockowitz Brooklyn, NY
I am not seeing any warning about changing a field's schema via https://www.drupal.org/docs/drupal-apis/update-api/updating-database-sch... β
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
liam morland β made their first commit to this issueβs fork.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Is this really supposed to be signed? Or is it that it's an easier change to make one signed instead of making the other unsigned?
- πΊπΈUnited States jrockowitz Brooklyn, NY
Is this really supposed to be signed? Or is it that it's an easier change to make one signed instead of making the other unsigned?
IDK, but I think the foreign sid in the
webform_submission_data
table should match the automatically defined sid in thewebform_submission
table. - π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Yes, they need to match for sure. Is it that Webform doesn't control the schema for table
webform_submission
? - πΊπΈUnited States jrockowitz Brooklyn, NY
Is it that Webform doesn't control the schema for table webform_submission?
As far as I understand, the webform_submission:sid column is created via the Entity API.
- πΊπΈUnited States jrockowitz Brooklyn, NY
I think we want to use Webform SVG.
https://git.drupalcode.org/project/webform/-/blob/6.3.x/icons/000000/web...