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

allow creating custom form extensions #63

Merged
merged 28 commits into from
Jan 27, 2025

Conversation

Rahien
Copy link
Collaborator

@Rahien Rahien commented Dec 19, 2024

Description

Allows the creation of custom form extensions by adding fields to forms and removing fields that have been added in this way (only from custom forms)

How to test

see lblod/frontend-lokaal-mandatenbeheer#449

controllers/form-definitions.ts Outdated Show resolved Hide resolved
controllers/form-definitions.ts Outdated Show resolved Hide resolved
domain/data-access/form-repository.ts Show resolved Hide resolved
services/custom-forms.ts Show resolved Hide resolved
app.ts Show resolved Hide resolved
@Rahien Rahien force-pushed the karel/lmb-333-create-form-extensions branch from e4117b1 to 3d9a0f5 Compare January 7, 2025 14:50
@Rahien Rahien requested a review from JonasVanHoof January 7, 2025 14:53
Comment on lines 108 to 319
INSERT DATA {
${sparqlEscapeUri(uri)} a form:Field;
ext:extendsGroup ${sparqlEscapeUri(fieldGroupUri)};
sh:name ${sparqlEscapeString(name)};
form:displayType ${sparqlEscapeUri(fieldDescription.displayType)};
sh:order ${sparqlEscapeInt(fieldDescription.order || 99999)};
sh:path ${sparqlEscapeUri(fieldDescription.path || generatedPath)};
mu:uuid ${sparqlEscapeString(id)}.
${sparqlEscapeUri(formUri)} form:includes ${sparqlEscapeUri(uri)}.
}
Copy link
Contributor

Choose a reason for hiding this comment

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

if we add validateBy here could we than pass on form validations? :D

@Rahien Rahien force-pushed the karel/lmb-333-create-form-extensions branch from 6a65bd0 to 3799065 Compare January 24, 2025 10:53
@Rahien Rahien merged commit 9a862ba into master Jan 27, 2025
1 check passed
@Rahien Rahien deleted the karel/lmb-333-create-form-extensions branch January 27, 2025 13:36
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.

2 participants