fix(handbook): Adjust FlowFuse Expert onboarding procedure - #5333
Conversation
ppawlowski
changed the title
✅ Deploy Preview for flowforge-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
dimitrieh
left a comment
dimitrieh
left a comment
There was a problem hiding this comment.
LGTM.
@ppawlowski @Steve-Mcl I take it that the insights mode config is still needed to be added in case they don't upgrade to the point release?
| assistant: | ||
| enabled: true | ||
| service: | ||
| url: https://expert.flowfuse.com/v1/openai | ||
| token: <Assistant Token> | ||
| expert: | ||
| enabled: true | ||
| service: | ||
| url: https://expert.flowfuse.com/v4/expert | ||
| token: <Expert Token> | ||
| The feature is enabled by adding valid tokens to the `.env` file used to configure a FlowFuse Platform. | ||
| Tokens should be added to the `ASSISTANT_TOKEN` and `EXPERT_TOKEN` environment variables (find and replace the `<Assistant Token>` and `<Expert Token>` placeholders with the actual tokens provided by FlowFuse Support/Sales). | ||
|
|
||
| ```yaml | ||
| ASSISTANT_TOKEN="<Assistant Token>" | ||
|
|
||
| EXPERT_TOKEN="<Expert Token>" |
There was a problem hiding this comment.
I am not privy to all changes made - I assume the relevant YML being removed here is reconstituted from the presence of these 2 tokens?
There was a problem hiding this comment.
This is correct - configuration is gated behind tokens.
| service: | ||
| url: https://expert.flowfuse.com/v1/openai | ||
| token: Provided-Assistant-Token | ||
| expert: |
There was a problem hiding this comment.
Are we not adding info to enable insights?
There was a problem hiding this comment.
Actually, thinking about that - I believe @cstns made a change in forge to default to enabled (can you confirm)?
There was a problem hiding this comment.
I think this was merged indeed, so if the client updates to the latest version of FlowFuse I assume this wouldn't be necessary.
There was a problem hiding this comment.
Actually, thinking about that - I believe @cstns made a change in forge to default to enabled (can you confirm)?
Confirming this was resolved in FlowFuse/flowfuse#7773. Note that this introduces a dependency between the configuration and the FlowFuse version: the setting only takes effect from the release (or minor release) that includes the fix onward.
5 participants
Description
This pull request adjusts the FlowFuse Expert onboarding process.
Requires FlowFuse/docker-compose#352 to be merged first.
Related Issue(s)
Closes https://github.com/FlowFuse/engineering/issues/141
Checklist