Skip to content

Conversation

@TejInaco
Copy link
Contributor

@TejInaco TejInaco commented Dec 29, 2025

This PR will:

Fix the functionality of adding forms for every type of property, even when the form uploaded by the user has an undefined value. This means there is no operation value associated with a given href value.
Previously, the application crashed when there was a form with an href value and no operation associated.

Users can add a form that must have both an operation value and an href value.
If the user adds a form with an operation already used for the same href value, it will raise a duplication error.

Closes #168

@netlify
Copy link

netlify bot commented Dec 29, 2025

Deploy Preview for editdor ready!

Name Link
🔨 Latest commit 7e1a150
🔍 Latest deploy log https://app.netlify.com/projects/editdor/deploys/6952a6233d3aac0008bdfbf8
😎 Deploy Preview https://deploy-preview-176--editdor.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
Signed-off-by: Ricardo M G da Silva <rephyrus0877@protonmail.com>
@TejInaco TejInaco force-pushed the fix-add-forms-for-affordances branch from c284f30 to 7e1a150 Compare December 29, 2025 16:02
@egekorkan egekorkan merged commit 72623d9 into eclipse:master Dec 29, 2025
9 checks passed
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.

Adding forms for affordances does not work

2 participants