Print enkel message. Niet de gehele JSON.

PeterC said: Check of de vorm van het error object consistent is.

Keyboard niet getoond wanneer zoekscherm is geladen (postcode input is wel gefocust)

https://acc.top.amsterdam.nl/cases/2008078_3?modal.fraud_prediction=1

Na het invullen van de postcode wordt de zoekknop al klikbaar. Dit zou pas moeten gebeuren als huisnr ook gevuld is.

• Backend: aanpassen fieldname " video-call' naar 'intercom' (@petercuret / @xavierbloemen )
• Frontend: testen Android/iOS (bevindingen in Bugs-kolom graag)
• Frontend: opschonen code
• Backend: Volledig overschrijven van data bij editen

Knop om privacymodus aan- en uit te zetten zit nu 'verborgen' in detailpagina (eerder waren alleen hier toezichthoudernamen te zien). Echter er is een 'overall plek' nodig (bv. in navigatie) voor hide/show privacy knop. De namen staan nl nu ook in de looplijst en in het lijstgeneratiescherm.

Hoe tonen we de gemaakte notities op detailpagina in Top-app, en wat gebeurt er eigenlijk met de andere 'notities' als kladblok BWV, logboek etc.

• Verwerken feedback TH

Als issue is toegevoegd via open issuemeldingen komt er geen melding: 'In mijn lijst', zoals dit wel gebeurt bij andere adressen die worden toegevoegd.

Heren, graag check of deze story nog relevant is:

We gebruiken nu de font settings uit Amsterdam Styled Components. Kan zijn dat "Avenir next" niet is geinstalleerd binnen Citrix en dat daardoor de fallback font "Arial" gebruikt wordt.

Yvonne en Mexime 22 april getest:

TOP Pakt fonts Avenir niet:
IE binnen ADW vanuit huis
Firefox binnen ADW vanuit huis
Chrome binnen ADW vanuit huis

Echter: Avenir komt wel door in www.amsterdam.nl zelf (zie screenshot), dus iets klopt er niet.

de titles van de stadia staan niet allemaal rechts uitgelijnd. zodra deze wat langer is, komt deze op nieuwe regel, terwijl er wel plek lijkt te zijn.

• Titels en hints omdraaien. Label is de vraag dik zwarte gedrukt met evt. (niet verplicht). hint is grijs eronder met toelichting op de vraag. (in praktijk dus teksten veelal omdraaien)

• radio buttons comform Primairy button component (blauw ipv zwart wanneer selected)

zaak detail informatie, incl. anchor en detail links. Max-width op lopende tekst op desktop zou de leesbaarheid verbeteren. Volgens mij klopt de line-height hier ook niet.

@gilleswittenberg says: Als afvinken "mislukt" blijft spinner draaien

Op dit moment komen gebruikers na het inloggen altijd uit op de 'mijn looplijsten' pagina. Het zou gebruiksvriendelijker zijn als de gebruiker weer op pagina terugkomen waar ze vandaan kwamen.

• automatically update npm dependencies script (https://github.com/dependabot) (to acc -> prod)
• do not install npm dev dependencies in Docker Image
• more, better unit testing
• use TypeScript Optional Chaining, Nullish Coalescing (https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-optional-chain.md)
• expand ESLint settings
• extract src/pages/LoginCallback.tsx logic to seperate components (vraag: niet beter package for Datapunt ipv elke keer 'wiel uitvinden').
• more abstract global state, fetching, error handling (useRestReducer), polling
• React-Final-Form for all forms
• use React Portal for global alert / message system
• seems assets (images, svgs) are being reloaded when components renderer
• all smaller @TODO in codebase (33 currently)
• split up TypeScript declaration files
• clean up QueryParams, routing

Opmerking is: 'Gek dat mijn lijst weer bovenaan begint en niet op de plek komt van het adres dat ik de notitie heb toegevoegd.'

Zaaksysteem --> Zoeken op eigenaar moet mogelijk zijn, vervolgens ook gelinkt aan een project.

• Invoer wizard
• Terugknoppen
• Starttijd automatisch invullen
• Starttijd 24 uurs klok
  • Hangt af van hoe je je OS hebt ingesteld. Even testen hoe het er op iOS/Android uitziet.
• Tijdinvoer verplicht
• Frontend: Starttijd + adres in balk onder titelbalk
• Post resultaat naar API (icm @petercuret )
• Checkbox in kaartje weghalen
• Notitie knop veranderen in rode knop
• Wanneer POST naar backend gelukt is, kaartje updaten
• Na opgeslagen notitie, alsnog kunnen wijzigen

Bij #302 gevonden bugs:

• /zoeken en /issuemeldingen should also be accessible when the user has not generated a "lijst", yet. Page https://acc.top.amsterdam.nl/lijst/nieuw/ should contain navigation links to both.
• https://acc.top.amsterdam.nl/lijst/923/zoeken/. Clearing search form sets error message "Dit veld is verplicht" for empty postal code input.
• Page content (header, navigation) is shown when user / access token is expired, before throwing alert.
• It's not possible to delete a "notitie" anymore. An empty "notitie" shows a message "Dit veld is verplicht"
• 404 page is gone
• https://acc.top.amsterdam.nl/lijst/nieuw/ is always accessible. (Also when the user has already generated one or more lists) This was placed behind a query parameter flag ?generate=1 before.

Dependabot encountered the following error when parsing your .dependabot/config.yml:

The property '#/update_configs/0' did not contain a required property of 'directory'

Please update the config file to conform with Dependabot's specification using our docs and online validator.

Het moet mogelijk zijn om (veilig) te documenten, workflow, status te delen uit het systeem (mailfunctionaliteit/deellink?)

MAJOR CLEANUP

Decided to execute a major cleanup after I had to modify a lot of boilerplate code (managing state) to implement the new feature "visit wizard", which I think will only increase whenever we add more features to TOP.

Hope you're not too overwhelmed by it :-).

How to check:

Please checkout "feature/major-clean-up", run a npm i, and then npm run start to review.

Pros

1. Folder structure same as ZaakSysteem

• Easy to reuse concepts in both repositories
• One way of working.
• Split code into features:
  • cases
  • itineraries
  • login
  • settings
  • shared
  • visits

2. Typesafe routes

Same solution as Zaaksysteem - frontend

3. Less code

~25% less lines of code in ts and tsx files. (Approx. 2000 Lines of code less)

4. Use generated types from Swagger schema.

As much as possible. (Not all responses are documented correctly).
When the backend is ready, switching to it completely is not that hard anymore.

5 Reusable ItineraryItemCard/ItineraryItemCardList

• Introduced a generic ItineraryItemCard/ItineraryItemCardList used by:
  • itineraries
  • suggestions
  • open-issues
  • start-address

6 Introduced environment variables

No runtime checks anymore

7 Replaced useGlobalState with useApiRequest

• Added options for useApiRequest:
  - lazy (boolean): When true: don't fetch data on mount.
  - example: useItineraries({ lazy: true })

• Added options for execDelete, execPost, execPut, execPatch:

  • skipCacheClear (boolean): When true, cache won't get cleared
    - example: const { execPatch } = useItineraries({ lazy: true }); execPatch({ skipCacheClear: true })
  • useResponseAsCache (boolean). When true, cache will be populated with the response of this request.
    - example: const { execPatch } = useItineraries({ lazy: true }); execPatch({ useResponseAsCache: true })

• Added posibility to update cache manually: eg:

const { updateCache } = useItineraries(); updateCache(( itineraries ) => {
// do something with itineraries 
})

• Easier to reason with, as latest API response is automatically fetched and used, there is no need to update state manually.

Cons:

• LOTS of changes
• Spinner is shown shortly when adding/deleting an item to a list. (Because cache is being cleared, and we refetch again automatically).
  However, dragging items around wont trigger a spinner as we can safely update the cache here.

Dependabot couldn't parse the package-lock.json found at /package-lock.json.

The error Dependabot encountered was:

Dependabot::DependencyFileNotParseable

View the update logs.

Als alles afgevinkt is kun je nog klikken op 'bekijk op Google Maps'

Openstaande topics notitie module:

Overall design

• design header in stappen 'verwerken bezoek en adres'- zie sketch design (andere scheidingslijn en kleiner font?)

• vraagteken styleguide comform maken

• opgeslagen: adres deel uitvagen zie sketch design

Geen Medewerking

• stap 1, subtitel: "Je kreeg geen medewerking. Zijn er zaken die verder opvielen? (niet verplicht)"

Geen medewerking en geen toegang verleend:

• stap 2: subtitel: "Wanneer kan dit adres het beste opnieuw bezocht worden?" (eerste zinnetje weglaten)
• Stap 2: volgorde radiobuttons: overdag, weekend, 's Avonds, onbekend
• Stap 2: titel open veld: "Toelichting: (niet verplicht)"
• stap 3: radio buttons eerst ja en dan nee
• Stap 3, Ja: zijn er nog noemenswaardigheden? (niet verplicht)
• Stap 3, Nee: weglaten "volgende stap", alleen "Waarom niet?"
• Stap 3: tekst pop-up:
• titel "Tegenhouden of doorlaten?
• "Tegenhouden (nee): het adres kan niet direct worden uitgezet. Voorbeeld: machtiging vereist."
• "Doorlaten: adres komt zonder tussenkomst weer in een looplijst terecht"

Kan pulldown menu van opties niet wegklikken door ernaast te klikken, alleen maar door weer op opties te drukken. (pijltje zou weer omhoog moeten staan, als teken dat je kan inklappen)

Gekke zin: toevoegen: je looplijst ipv lijst

image

Handig: lijst sorteren? op % of afstand?

Lineheight lijkt dikker dan voorbeeld. Hoe kan ik kleur testen?

Zodat we overzicht hebben van de lijsten die op een dag gelopen dienen te worden. In ieder geval overzicht voor development, productowner, data mensen (niet per sé voor planner, alhoewel hij de beoordeling wel goed kan maken). (deels voor kwaliteitsdoeleinden)

BV: Ik bedacht me dit net. Het voorbereidend onderzoek bij een melding heeft ook nog aardig wat potentieel om te versnellen / moderniseren. Wellicht kan die ook nog worden meegenomen ;-)

MP: Yeah, maar we gaan de applicaties met elkaar laten praten dus daar hebben we overigens wel al een aardige stap in zometeen: SIA geeft een deugdelijke melding met info weer, ten behoeve van het aanvullend onderzoek: TOP (nu alleen bij aangemaakte zaak informatie te vinden, maar dan niet gekoppeld aan zaak maar aan adres bronnen zijn immers al gekoppeld, zodat projectmedewerker info top 1 bron heeft om aan te vullen, en evt. andere openstaande zaken die TOP dan weergeeft).

Hier ligt de winst vooral bij het vinden bij internetadvertenties in het vooronderzoek. (bijvoorbeeld scrapelijsten 2.0). Bij openen zaak wordt dan automatisch gezocht in de scrapelijst met de potentiele match van de internetadvertentie bij dit adres waar een zaak wordt aangemaakt. Dat dit geen (zo min mogelijk) mankracht meer kost en het dus niet (advertentie bij adres koppelen) als project hoeft te worden gedaan door allemaal collega’s. Dit kan je meenemen.

Aanpassing obv wensen projectmewerker. Toevoegen meerdere postcoderanges én aanpassing begindatum naar begindatum van het nieuwste stadium. zie UX in bijgevoegde PDF.
070620 - UX Invoer meerdere Postcoderanges + Begindatum.pdf

We geven geen melding als adres al in gebruik is, dan staat 'geen resultaten'. Wellicht ook toevoegen bericht van 'toevoegen gelukt'.

@gilleswittenberg Notitie is niet meer te deleten. Als alle characters verwijderd worden is form invalid.

• Concept Notities in looplijst houden (ipv op detailpagina) huidige ontwerp niet weggoien, andere sprint!
• Notities uberhaupt in ZKS?

Dependabot couldn't parse the package-lock.json found at /package-lock.json.

The error Dependabot encountered was:

Dependabot::DependencyFileNotParseable

View the update logs.

Story: Als toezichthouder wil ik een zaak verwijderen uit mijn zelf gegenereerde looplijst, zodat ik geen zaken loop waar ik een negatieve ervaring heb

Comment: Toezichthouder kan zaak verwijderen (bijv. conflict, eerdere nare ervaring) voor veiligheid, gemoedsrust, werkplezier of zelfstandigheid.

Voor later:

• max 3 verwijderen
• toevoegen reden van verwijderen (voor management en om cases te identificeren die niet uitgezet moeten kunnen worden).

Als je een adres zoekt voordat je een looplijst hebt gemaakt kan je wel adres zoeken maar niet toevoegen (logisch). Zou logisch zijn om dan optie aan te bieden, maak startadres.

As a PM i would like te see the loction of the TZH so oI can inform him of nearby issues

#notitie: karakter teller bij max aantal karakters notitie toevoegen

zie: https://designsystem.amsterdam.nl/7awj1hc9f/p/657629-text-input/i/6632402

Waarom wordt google maps in nieuwe tab geopend bij overzicht en in zelfde scherm vanaf detail pagina?

Voorstel: altijd in nieuwe tab (of naar google-maps-app, wanneer geïnstalleerd)

border lijkt 10 pixels ipv 15 voor adressen. vinkboxje staat wel op 15 afstand