Skip to content

MBS-10901: Allow the chatbot to work in agent mode with unavailable chat purpose - #72

Open
PhMemmel wants to merge 5 commits into
mainfrom
MBS-10901-Make_agent_available_despite_disabled_chat
Open

MBS-10901: Allow the chatbot to work in agent mode with unavailable chat purpose#72
PhMemmel wants to merge 5 commits into
mainfrom
MBS-10901-Make_agent_available_despite_disabled_chat

Conversation

@PhMemmel

@PhMemmel PhMemmel commented Jun 30, 2026

Copy link
Copy Markdown
Member

Closes #66

@lucaboesch

Copy link
Copy Markdown
Contributor

This branch has conflicts that must be resolved.
Do you like me taking over, @PhMemmel ?

@PhMemmel
PhMemmel force-pushed the MBS-10901-Make_agent_available_despite_disabled_chat branch from 9e6b3e0 to 30afdc3 Compare July 27, 2026 03:55
@PhMemmel

Copy link
Copy Markdown
Member Author

@lucaboesch Branch should be clean again. Thanks for spotting!

Comment thread templates/state_header.mustache
Comment thread templates/state_header.mustache Outdated
Comment thread amd/src/components/headeractions.js Outdated
@PhMemmel
PhMemmel force-pushed the MBS-10901-Make_agent_available_despite_disabled_chat branch from 30afdc3 to 7825918 Compare August 6, 2026 14:10
@PhMemmel

PhMemmel commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Thanks for your review:

  • I amended the small context typo in state_header in the latest commit
  • I provided the other changes in a fixup commit which I can rebase into the first commit if everything's fine.

@PhMemmel PhMemmel closed this Aug 6, 2026
@PhMemmel PhMemmel reopened this Aug 6, 2026
@PhMemmel
PhMemmel force-pushed the MBS-10901-Make_agent_available_despite_disabled_chat branch from 7825918 to e9c1f4b Compare August 6, 2026 14:21
@lucaboesch

Copy link
Copy Markdown
Contributor

If I'm not mistaken the mariadb lines should be

        image: mariadb:11.8
        env:
          MARIADB_ALLOW_EMPTY_ROOT_PASSWORD: "1"
          MARIADB_CHARACTER_SET_SERVER: utf8mb4
          MARIADB_COLLATION_SERVER: utf8mb4_unicode_ci
        ports:
          - 3306:3306
        options: --health-cmd="mariadb-admin ping" --health-interval 10s --health-timeout 5s --health-retries 3

because mysqladmin on that mariadb image is retired.

@PhMemmel
PhMemmel force-pushed the MBS-10901-Make_agent_available_despite_disabled_chat branch from e9c1f4b to 84727fb Compare August 11, 2026 05:24
@PhMemmel
PhMemmel requested a review from Snl-FGH August 11, 2026 05:31
@PhMemmel

Copy link
Copy Markdown
Member Author

If I'm not mistaken the mariadb lines should be

        image: mariadb:11.8
        env:
          MARIADB_ALLOW_EMPTY_ROOT_PASSWORD: "1"
          MARIADB_CHARACTER_SET_SERVER: utf8mb4
          MARIADB_COLLATION_SERVER: utf8mb4_unicode_ci
        ports:
          - 3306:3306
        options: --health-cmd="mariadb-admin ping" --health-interval 10s --health-timeout 5s --health-retries 3

because mysqladmin on that mariadb image is retired.

Thanks @lucaboesch , yes, we'll have to implement this gradually across all our pipelines unfortunately. I updated ci.yml according to the latest dist file from moodle-plugin-ci

@PhMemmel

Copy link
Copy Markdown
Member Author

@Snl-FGH I updated GHA as well and also pushed a fix commit for the mustache lint errors. I hope the patch is fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AI Chat window, when only agent function/purpose is enabled, should not be called "AI chat" but "AI agent"

3 participants