Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Conversation

@RXminuS
Copy link
Contributor

@RXminuS RXminuS commented Apr 16, 2024

Closes #59751

  • Added variant option for Notices. Defaults to default option for position.
  • Added styleOverrides option to Notices with three fields:
    • backgroundColor
    • textColor
    • textCentered

This means in the site-config a user can now specify notices as:

{
"notices": [
    {
      "message": "THIS IS AN IMPORTANT NOTICE",
      "location": "top",
      "variant": "danger",
      "styleOverrides": {
        "backgroundColor": "#7f1d1d",
        "textColor": "#fecaca",
        "textCentered": true
      }
    }
  ]
}

resulting in the following notice
CleanShot 2024-03-26 at 00 03 52@2x

Test plan

  • Updated existing regression tests
  • Manually tested with various site-config settings.

Backport ccba7d4 from #61338

… position. (#61338)

Closes #59751

Co-authored-by: Rik Nauta <rik.nauta@sourcegraph.com>
@RXminuS RXminuS self-assigned this Apr 16, 2024
@RXminuS RXminuS requested a review from a team April 16, 2024 11:19
@cla-bot cla-bot bot added the cla-signed label Apr 16, 2024
@sourcegraph-bot
Copy link
Contributor

📖 Storybook live preview

- GitHub apps installation records will only be deleted from the database if the GitHub App has been uninstalled or if the GitHub app has been deleted. [#60460](https://github.com/sourcegraph/sourcegraph/pull/60460)
- The Anthropic provider for Cody has been updated to use the messages API which includes support for Claude 3 models. This is applicable to both BYOK and Cody Gateway users. The messages API does not support model identifiers which only set a major model version such as: `claude-2`, `claude-instant-v1` and `claude-instant-1`. Default values have been updated to `claude-2.0` and `claude-instant-1.2`, any legacy models identifiers in the site config will be set to the corresponding default previously mentioned. [#60953](https://github.com/sourcegraph/sourcegraph/pull/60953) [#61324](https://github.com/sourcegraph/sourcegraph/pull/61324)
- The AWS Bedrock provider for Cody has been updated to use Anthropic's Messages API, bringing support for Claude 3 models. [#61347](https://github.com/sourcegraph/sourcegraph/pull/61347)
- Notices configured in the site config now allow for specifying a style or color. [#61338](https://github.com/sourcegraph/sourcegraph/pull/61338)
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't need to be here. It only need to exist on main.
I've added it here for you: https://github.com/sourcegraph/sourcegraph/pull/61918

@Chickensoupwithrice Chickensoupwithrice merged commit 6af7844 into 5.3.9104 Apr 18, 2024
@Chickensoupwithrice Chickensoupwithrice deleted the sg/backport-61338-to-5.3.9104 branch April 18, 2024 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants