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

Fix/375 asset select issues #379

Merged
merged 13 commits into from
Jan 22, 2025
Merged

Fix/375 asset select issues #379

merged 13 commits into from
Jan 22, 2025

Conversation

iehkaatee
Copy link
Contributor

voor #375

Copy link

vercel bot commented Jan 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
signalen-frontend-wcag ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2025 11:24am

@justiandevs
Copy link
Contributor

Wanneer ik in stap 1 de tekst "Vuurwerkoverlast" in typ staat er bij stap 2 direct onder de map "De locatie is gepind op de kaart", dit lijkt mij onlogisch? Want ik heb nog niks uitgevoerd op het gebied van het kiezen van een locatie. Denk dat dit verwarring kan veroorzaken.

Scherm­afbeelding 2025-01-20 om 11 21 26

@justiandevs
Copy link
Contributor

Er is nu iets ook helemaal kapot gegaan met de combobox in de AssetSelect, zodra je een asset hebt geselecteerd -> uit de dialoog gaat en dan weer in de dialoog gaat en toch een adres wil selecteren via de combobox toont deze geen suggesties en reset hij zichzelf telkens (de waarde in het inputveld van de combobox).

@justiandevs
Copy link
Contributor

Dit bovenste gaat volgens mijn browser console errors mis bij de API call naar de PDOK service, deze PDOK service geeft op zijn beurt de volgende error:

{
  "error":{
    "metadata":[
      "error-class","org.apache.solr.common.SolrException",
      "root-error-class","org.apache.solr.parser.ParseException"],
    "msg":"org.apache.solr.search.SyntaxError: Cannot parse 'gemeentenaam:()': Encountered \" \")\" \") \"\" at line 1, column 14.\nWas expecting one of:\n    <NOT> ...\n    \"+\" ...\n    \"-\" ...\n    <BAREOPER> ...\n    \"(\" ...\n    \"*\" ...\n    <QUOTED> ...\n    <TERM> ...\n    <PREFIXTERM> ...\n    <WILDTERM> ...\n    <REGEXPTERM> ...\n    \"[\" ...\n    \"{\" ...\n    <LPARAMS> ...\n    \"filter(\" ...\n    <NUMBER> ...\n    <TERM> ...\n    ",
    "code":400}}

@justiandevs
Copy link
Contributor

Scherm­afbeelding 2025-01-20 om 11 30 25

Misschien ook een idee om dit nog op te pakken in dit issue? Waarom gebruikt de lijst met assets niet de gehele hoogte wanneer er genoeg assets zijn om te tonen?

@justiandevs
Copy link
Contributor

Voor de rest kom ik zelf geen gekke dingen meer tegen.

@iehkaatee iehkaatee merged commit 8b327d7 into main Jan 22, 2025
8 checks passed
@iehkaatee iehkaatee deleted the fix/375-asset-select-issues branch January 22, 2025 11:54
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