Skip to content

Conversation

@sgort
Copy link

@sgort sgort commented Sep 3, 2025

Closes #15

Test available at https://owc-gemeente.test.open-regels.nl/formulier/
Related errors in console log are gone.

image

@mvdhoek1
Copy link
Collaborator

mvdhoek1 commented Sep 3, 2025

Hi @sgort ,

Dank voor je bijdrage!
Klopt het dat in jouw test de velden die ingevuld worden door de plugin niet read-only zijn gemaakt?

@sgort
Copy link
Author

sgort commented Sep 3, 2025

Ha @mvdhoek1 dat klopt, want ik was gewoon de prefilled velden ook nog zelf met de hand te kunnen wijzigen. Dat is nl. wel de bedoeling. Alleen, dat werkt na deze code aanpassing niet meer kom ik nu achter. Ik kan de velden partner_geslachtsnaam en kind_geboorteplaats bijv. niet meer aanpassen.

Als ik de wijziging in deze PR terugdraai dan gaat dat weer wel...

Guess deze PR negeren/afvoeren en dan moet ik me minder druk maken over de formulier/:3 Uncaught ReferenceError: jQuery is not defined at formulier/:3:5 errors in console.

@sgort
Copy link
Author

sgort commented Sep 3, 2025

Sorry voor de PR. Mijn gebrek aan kennis en blinde ambities zitten me hier in de weg. Ik moet de jQuery timing issues aan mijn eigen kant / in mijn eigen plugin oplossen.

@mvdhoek1
Copy link
Collaborator

mvdhoek1 commented Sep 4, 2025

@sgort ik vind dit een goede eerste stap, de jQuery melding in de console komt wel van deze plug-in namelijk.
Even het volgende kortsluiten eerst; we willen vooraf ingevulde velden niet meer kunnen laten wijzigen was het idee. Het heeft bijvoorbeeld geen zin om bijv. voornamen wijzigen in het formulier. Hoe sta jij hier in?

@sgort
Copy link
Author

sgort commented Sep 4, 2025

De ontwerpbeslissing (als dat zo heet) die ik in mijn formulier(en) wil maken, is dat de gebruiker ALTIJD regie houdt over de gegevens die van hem/haar bij de overheid geregistreerd staan. Zie het kunnen wijzigen van vooringevulde gegevens als een onderdeel van het Proces van terugmelden.

@mvdhoek1
Copy link
Collaborator

1 probleem, de meeste informatie komt uit de BRP en wij schieten daar niets in. De enige plek waar zo'n wijziging zichtbaar is zal dan onder de inzendingen zijn van GravityForms.

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.

Script run before jQuery is available

2 participants