Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat!: implementAI Bridge heading to/deployment/observability#20791

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
jakehwll wants to merge20 commits intomain
base:main
Choose a base branch
Loading
fromjakehwll/ai-bridge-observability

Conversation

@jakehwll
Copy link
Contributor

@jakehwlljakehwll commentedNov 17, 2025
edited
Loading

Caution

In whichever release this lands, we've removed the ability to provide keys via a YAML file (specifically onopenai_key,anthropic_key,bedrock_access_key and finallybedrock_access_key_secret). This will need to be described in the release notes as to not break peoples AI Bridge integrations upgrading from older versions.

This pull-request ensures that we can see the overview of the settings of theAI Bridge feature within the/deployment/observability route. This set of options only render when theaibridge feature flag is enabled.

Preview

preview-ai-bridge-observability

@matifali
Copy link
Member

@jakehwll, can you share a screenshot of what this will look like?

jakehwll reacted with heart emoji

@jakehwlljakehwllforce-pushed thejakehwll/ai-bridge-observability branch fromff41882 to51cb6b9CompareNovember 18, 2025 01:21
Copy link
Member

@matifalimatifali left a comment

Choose a reason for hiding this comment

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

Looks good 🔥

Copy link
Member

@mtojekmtojek left a comment

Choose a reason for hiding this comment

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

only nit-picks

@matifali
Copy link
Member

@jakehwll make sure we hide the API keys in UI. I see red boxes but they should not be visible.

dannykopping reacted with thumbs up emoji

Copy link
Member

@mtojekmtojek left a comment

Choose a reason for hiding this comment

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

Thanks!

@dannykoppingdannykopping changed the titlefeat: implementAI Bridge heading to/deployment/observabilityfeat!: implementAI Bridge heading to/deployment/observabilityNov 28, 2025
@github-actionsgithub-actionsbot added the release/breakingThis label is applied to PRs to detect breaking changes as part of the release process labelNov 28, 2025
@dannykopping
Copy link
Contributor

@jakehwll I added a! to the PR title to indicate a breaking change 👍

jakehwll reacted with heart emoji

Copy link
Contributor

@dannykoppingdannykopping left a comment
edited
Loading

Choose a reason for hiding this comment

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

LGTM, please ensure description screenshot is updated prior to merge

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

Reviewers

@matifalimatifalimatifali approved these changes

@dannykoppingdannykoppingdannykopping approved these changes

@jaaydenhjaaydenhjaaydenh approved these changes

@mtojekmtojekmtojek approved these changes

Assignees

@jakehwlljakehwll

Labels

release/breakingThis label is applied to PRs to detect breaking changes as part of the release process

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jakehwll@matifali@dannykopping@jaaydenh@mtojek

[8]ページ先頭

©2009-2025 Movatter.jp