Skip to content
Snippets Groups Projects
Unverified Commit c715206f authored by Nemanja Glumac's avatar Nemanja Glumac Committed by GitHub
Browse files

Use `updateUserSetting` for user-local settings (#39359)

* Treat `dismissed-browse-models-banner` as a user-setting

* Treat `dismissed-custom-dashboard-toast` as a user-setting
parent bfd6895f
Branches
Tags
No related merge requests found
import { useState } from "react";
import { t } from "ttag";
import { updateSetting } from "metabase/admin/settings/settings";
import { useDispatch, useSelector } from "metabase/lib/redux";
import { updateUserSetting } from "metabase/redux/settings";
import { getSetting } from "metabase/selectors/settings";
import { Flex, Paper, Icon, Text } from "metabase/ui";
......@@ -19,7 +19,7 @@ export function ModelExplanationBanner() {
const dismissBanner = () => {
setShouldShowBanner(false);
dispatch(
updateSetting({
updateUserSetting({
key: "dismissed-browse-models-banner",
value: true,
}),
......
......@@ -2,7 +2,6 @@ import { useEffect } from "react";
import { replace } from "react-router-redux";
import { t } from "ttag";
import { updateSetting } from "metabase/admin/settings/settings";
import {
useDashboardQuery,
useDatabaseListQuery,
......@@ -11,6 +10,7 @@ import {
import LoadingAndErrorWrapper from "metabase/components/LoadingAndErrorWrapper";
import { useDispatch, useSelector } from "metabase/lib/redux";
import { canUseMetabotOnDatabase } from "metabase/metabot/utils";
import { updateUserSetting } from "metabase/redux/settings";
import { addUndo } from "metabase/redux/undo";
import { getSettingsLoading } from "metabase/selectors/settings";
import type Database from "metabase-lib/metadata/Database";
......@@ -112,7 +112,7 @@ const useDashboardPage = () => {
icon: "info",
timeout: 10000,
actions: [
updateSetting({
updateUserSetting({
key: "dismissed-custom-dashboard-toast",
value: true,
}),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment