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

Handle HTTP errors in links #9

Conversation

BuonDavide
Copy link
Contributor

Aggiunge la gestione degli errori nella parte links del chatbot.

  • Se il link è indicizzato ma ha il campo http_error valorizzato, allora si visualizza con uno sfondo arancione ed il messaggio di errore
  • Aggiunto in alto a destra un menu dropdown per filtrare i link visualizzati (All links, Indexed Links, Non Indexed Links e Error Links)
  • Se il link si trova in http http_error compare, al posto del pulsante dei metadati, un tasto per reindicizzare ( che richiama POST /api/brevia/index/link)

@@ -70,6 +70,7 @@
"FILE_TYPE_MUST_BE": "Accepted file type:",
"FILE_TYPE_NOT_ACCEPTED": "File type not supported",
"FILES": "Files",
"FILTERS": "Filters",
Copy link
Contributor

Choose a reason for hiding this comment

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

Intstead of Filters I would use Show as a label

options: {},
},
});
indexing.value = false;
Copy link
Contributor

Choose a reason for hiding this comment

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

after a reindex we should read the indexed document in order to see if a document has been actually indexed/updated or not

We should call GET /api/brevia/index/{collection_id}/{document_id} or GET /api/brevia/index/{collection_id}/document_metadata?document_id={document_id}

A user feedback is then needed + change of state (like http_error removal) if something has changed

@stefanorosanelli stefanorosanelli changed the title Gestione errori http in links Handle HTTP errors in links Jul 29, 2024
@stefanorosanelli stefanorosanelli marked this pull request as ready for review July 29, 2024 15:35
@stefanorosanelli stefanorosanelli merged commit dff6fa0 into brevia-ai:main Jul 30, 2024
@BuonDavide BuonDavide deleted the issue/190-gestione-errori-http-in-links branch September 4, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants