Skip to content

chore: remove notifications experiment #14869

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Oct 1, 2024
Merged

chore: remove notifications experiment #14869

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5cc1e73
chore: remove notifications experiment
dannykopping Sep 30, 2024
cb0be34
Pass transaction into report generator to avoid deadlock
dannykopping Sep 30, 2024
6e7c919
Only close done channel when notifier is present
dannykopping Sep 30, 2024
7c571ae
Reorder notification manager to avoid race with CreateInMemoryTaggedP…
dannykopping Sep 30, 2024
0107ad1
Remove experiment reference & show beta badge in frontend
dannykopping Oct 1, 2024
0aa6d7b
Ensuring done channel cannot be closed twice
dannykopping Oct 1, 2024
bf1a148
Beta tag on account notification settings pages
dannykopping Oct 1, 2024
27d28b3
make fmt
dannykopping Oct 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Beta tag on account notification settings pages 
Signed-off-by: Danny Kopping <danny@coder.com>
  • Loading branch information
@dannykopping
dannykopping committed Oct 1, 2024
commit bf1a148f3e50e7ba164fdd051e38a16dd9c498c3
2 changes: 1 addition & 1 deletion site/src/pages/ManagementSettingsPage/SidebarView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ const DeploymentSettingsNavigation: FC<DeploymentSettingsNavigationProps> = ({
Users
</SidebarNavSubItem>
)}
<Stack direction={"row"} alignItems={"center"} css={{ gap: 0 }}>
<Stack direction="row" alignItems="center" css={{ gap: 0 }}>
<SidebarNavSubItem href="notifications">
Notifications
</SidebarNavSubItem>
Expand Down
1 change: 1 addition & 0 deletions site/src/pages/UserSettingsPage/NotificationsPage/NotificationsPage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,7 @@ export const NotificationsPage: FC = () => {
title="Notifications"
description="Configure your notification preferences. Icons on the right of each notification indicate delivery method, either SMTP or Webhook."
layout="fluid"
featureStage="beta"
>
{ready ? (
<Stack spacing={4}>
Expand Down
3 changes: 2 additions & 1 deletion site/src/pages/UserSettingsPage/Sidebar.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import {
import { UserAvatar } from "components/UserAvatar/UserAvatar";
import { useDashboard } from "modules/dashboard/useDashboard";
import type { FC } from "react";
import {FeatureStageBadge} from "components/FeatureStageBadge/FeatureStageBadge";

interface SidebarProps {
user: User;
Expand Down Expand Up @@ -58,7 +59,7 @@ export const Sidebar: FC<SidebarProps> = ({ user }) => {
Tokens
</SidebarNavItem>
<SidebarNavItem href="notifications" icon={NotificationsIcon}>
Notifications
Notifications <FeatureStageBadge contentType="beta" size="sm" />
</SidebarNavItem>
</BaseSidebar>
);
Expand Down
Loading