Skip to content

Conversation

@mcncl
Copy link
Contributor

@mcncl mcncl commented Mar 26, 2024

We don't need to have the functionality yet, but the initial field could be set on first config right now.

We'll need to check for its existence to replace the empty string in the future.

I can see a future where each org may have its own token, but at the moment a top level makes sense to me; we can always change it later.

Copy link
Contributor

@jradtilbrook jradtilbrook left a comment

Choose a reason for hiding this comment

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

👌

@mcncl mcncl enabled auto-merge (squash) March 26, 2024 02:37
@mcncl mcncl merged commit a33b0e9 into 3.x Mar 26, 2024
@mcncl mcncl deleted the bm/ai_token_config branch March 26, 2024 02:39
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.

2 participants