Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Adds filter change to the chatbot #1494

Merged
merged 6 commits into from
Oct 2, 2023
Merged

Adds filter change to the chatbot #1494

merged 6 commits into from
Oct 2, 2023

Conversation

mamikals
Copy link
Contributor

@mamikals mamikals commented Sep 28, 2023

Oppsummering av hva som er gjort

Legger til mulighet å endre chatbot filtre fra boost. Det vil da si at AI ekspertene som jobber med intentene i Boost kan la brukeren endre filtre ved knapper i stedet for nedtrekksmeny, slik det er nå. Dette er hovedsakelig for å enkelt overføre personer til arbeidsgiver chat, hvis de åpnet den i privatperson context.
Satt også opp automatisk valg av arbeidsgiver hvis personen er på en arbeidsgiver-side når chatbotten starter. Og det samme for nynorsk, hvis de er på en nynorsk side. (Det finnes per i dag ingen nynorsk arbeidsgiver filter i boost, så det fungerer bare for privatperson).

Testing

Testet det både lokalt, og deployet til dev og testet der.

Dette trenger jeg å få et ekstra blikk på

Jeg gjorde noen ekstra tester og fant ut at det automatisk filteret ikke fungerte helt perfekt i alle tilfeller. Spesifikt så settes det bare når chatbotten instansieres, og vil dermed alltid default til det filteret hver gang en ny chat startes, uavhengig av hva brukeren har valgt på chatten før eller hvilken side de er på når den nye chatten startes. Dette hadde jeg antatt var ønskelig, men med litt videre testing fant jeg noen problemer med at nav.no siden er automatisk privatperson - bokmål, som gjorde funksjonaliteten veldig lite brukbar og ikke særlig intuitiv.

Skjermbilde hvis relevant

Copy link
Contributor

@terjeofnorway terjeofnorway left a comment

Choose a reason for hiding this comment

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

Hadde én kommentar bare. Dessuten, kan du oppdatere beskrivelsen i PR'en? Den teksten som er der nå er bare eksempel-tekst :)

src/csp.ts Outdated Show resolved Hide resolved
@mamikals
Copy link
Contributor Author

Det kan jeg godt 😄, jeg la merke til noen ekstra problemer siden jeg skulle oppdatere beskrivelsen. Så jeg kommer til å ta bort noen av endringene, for så å diskutere det med teamet om de ønsker å ha det slikt.
Men resten av endringene kan godt merges inn, de er ønskelige fortsatt.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@terjeofnorway terjeofnorway merged commit ff4be44 into master Oct 2, 2023
3 checks passed
@terjeofnorway terjeofnorway deleted the chatbotFilters branch October 2, 2023 10:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants