Skip to content

FEATURE: Add "id" to field and label#18

Merged
mficzel merged 1 commit intomainfrom
task/addIdToFields
Nov 17, 2023
Merged

FEATURE: Add "id" to field and label#18
mficzel merged 1 commit intomainfrom
task/addIdToFields

Conversation

@mficzel
Copy link
Copy Markdown
Member

@mficzel mficzel commented Oct 5, 2023

the field is prefixed with "field_" to always start with a valid id as is required by spec.

Resolves: #12

the field is prefixed with "field_" to always start with a valid id as is required by spec.

Resolves: #12
Copy link
Copy Markdown
Contributor

@jonnitto jonnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine for me, but the composer change has nothing to do with this PR

@mficzel
Copy link
Copy Markdown
Member Author

mficzel commented Oct 6, 2023

but the composer change has nothing to do with this PR

yes those are a minor cleanup on the way.

@mficzel
Copy link
Copy Markdown
Member Author

mficzel commented Oct 6, 2023

We should merge neos/fusion-form#86 first and i assume it will fix the issue aswell.
Will convert this to a draft for now.

@mficzel mficzel marked this pull request as draft October 6, 2023 14:36
@mficzel mficzel marked this pull request as ready for review November 17, 2023 14:30
@mficzel mficzel merged commit d55b0ac into main Nov 17, 2023
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.

Always start identifiers with an alphabetic character

2 participants