Skip to content
Snippets Groups Projects

Vue router for AI Agents

Merged Darby Frey requested to merge update-ai-agent into master

What does this MR do and why?

The AI Agents section was originally set up with separate Rails controller actions for each page, and separate Vue apps to load the UI on each page. This MR brings the interface together into a single Vue app that uses the Vue router to move between interfaces. See the gif below for a preview of the UI.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

2024-02-02_17.07.42

How to set up and validate locally

  1. Ensure you have an EE license activated

  2. Enable the feature flag

    Feature.enable(:agent_registry)
  3. Navigate to <project>/-/ml/agents and ensure that the empty state is loading correctly

  4. Click on 'Create agent'

  5. Create an agent (name cannot have spaces, prompt can be anything)

  6. Navigate to <project>/-/ml/agents and ensure the agent list shows up with the item that was just created

Edited by Darby Frey

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Darby Frey added 1 commit

    added 1 commit

    • c9fb6113 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Darby Frey requested review from @fernando-c and @alexbuijs

    requested review from @fernando-c and @alexbuijs

  • Darby Frey added 1 commit

    added 1 commit

    • 791615fa - Updated translations and graphql docs

    Compare with previous version

  • Alex Buijs
  • Alex Buijs
  • Alex Buijs
  • Alex Buijs
  • Alex Buijs removed review request for @alexbuijs

    removed review request for @alexbuijs

  • Darby Frey added 1 commit

    added 1 commit

    • 00070700 - Updated translations and graphql docs

    Compare with previous version

  • Darby Frey added 2 commits

    added 2 commits

    • bcddb349 - Reorganizing code, removed unused files
    • f9bd23e2 - Updated javascript tests

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • Darby Frey added 1 commit

    added 1 commit

    Compare with previous version

  • mentioned in issue #438763 (closed)

  • Darby Frey requested review from @alexbuijs

    requested review from @alexbuijs

  • Alex Buijs approved this merge request

    approved this merge request

  • Alex Buijs requested review from @splattael and removed review request for @alexbuijs

    requested review from @splattael and removed review request for @alexbuijs

  • Peter Leitzen approved this merge request

    approved this merge request

  • added GraphQL label

  • Peter Leitzen mentioned in issue #437540

    mentioned in issue #437540

  • Darby Frey added 10 commits

    added 10 commits

    • 5298760d - Vue router for AI Agents
    • 5b80d063 - Apply 1 suggestion(s) to 1 file(s)
    • e267c69a - Updated translations and graphql docs
    • 6b5c089c - Updated translations and graphql docs
    • 183f14f1 - Reorganizing code, removed unused files
    • dc1862cf - Updated javascript tests
    • 37b5f0e7 - Updated ruby tests
    • d840d262 - Updated externalizations
    • 482fcd69 - Fixing presenter test
    • 32fd855f - Update permissions and tests for AI Agents controller

    Compare with previous version

  • Darby Frey reset approvals from @splattael by pushing to the branch

    reset approvals from @splattael by pushing to the branch

  • Darby Frey requested review from @splattael

    requested review from @splattael

    • Resolved by Fernando Cardenas

      @darbyfrey great work on unifying the seperate vue apps from seperate rails vue into one app that is client side routing enabled.

      The code looked good to me, only had one question.

      question (non-blocking)

      Navigate to <project>/-/ml/agents and ensure that the empty state is loading correctly

      How do we navigate to the page? Is there a nav menu item behind a feature flag? I couldn't find it so I had to manually enter the URL in the browser.

  • Fernando Cardenas approved this merge request

    approved this merge request

  • Peter Leitzen approved this merge request

    approved this merge request

  • Darby Frey requested review from @arfedoro

    requested review from @arfedoro

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading