Fix links in training analytics - Githubissues

buerokratt / Buerokratt-Chatbot

1 stars 17 forks source link

Fix links in training analytics #822

Open jaX10bt opened 1 month ago

jaX10bt commented 1 month ago

AS A Backoffice User I WANT TO see all the intents in a model and be able to go to the intent examples page by clicking "Mine näitesse" SO THAT I can add/delete/modify intent examples

Currently in /training/analytics/overview clicking "Mine näitesse" on a common intent links to common intents view which is hidden. Clicking "Mine näitesse" on a regular intent (not common) links to intents view but from there it is not possible to go to another intent. How it should be: Clicking "Mine näitesse" on any intent in the intents overview page links to intent exaples page for the selected intent.

Acceptance Criteria

[x] Clicking "Mine näitesse" in /training/analytics/overview takes the user to the intent examples for the selected intent in /training/training/intents
[x] After going to /training/training/intents by clicking "Mine näitesse" it is possible to navigate to another intent

Fossur commented 1 month ago

https://github.com/buerokratt/Training-Module/pull/629

turnerrainer commented 1 month ago

@SanderSepp takes over.

turnerrainer commented 1 month ago

buerokratt/Training-Module#629 Obsolete, resolved within other tasks.

turnerrainer commented 1 month ago

Resolved as part of https://github.com/buerokratt/Buerokratt-Chatbot/issues/822

joonasroosalung commented 1 month ago

After https://github.com/buerokratt/Training-Module/pull/626 is merged: PR https://github.com/buerokratt/Training-Module/pull/632

turnerrainer commented 1 month ago

Finished by @joonasroosalung , blocked by @turnerrainer

turnerrainer commented 1 month ago

@joonasroosalung blocker resolved (https://github.com/buerokratt/Training-Module/pull/626 merged), please proceed.

joonasroosalung commented 1 month ago

@turnerrainer PR ready for merge https://github.com/buerokratt/Training-Module/pull/632

jaX10bt commented 1 month ago

Clicking "Mine näitesse" should take the user to /training/training/intents rather than /training/training/common-intents

joonasroosalung commented 1 month ago

@jaX10bt Previously regular and common intents have been differentiated by name and they linked to whichever page they were from. Now it should always link to the regular intents page?

joonasroosalung commented 1 month ago

PR https://github.com/buerokratt/Training-Module/pull/643