Skip to content
Snippets Groups Projects
Unverified Commit 129518fc authored by Alexander Polyankin's avatar Alexander Polyankin Committed by GitHub
Browse files

Improve slack settings messages (#25379)

parent 25202443
No related branches found
No related tags found
No related merge requests found
......@@ -20,7 +20,7 @@ const SlackSetupForm = ({ onSubmit }: SlackSetupFormProps): JSX.Element => {
<FormField name="slack-app-token" />
<FormMessage>
{t`Finally, open Slack, create a public channel and enter its name below.`}{" "}
{t`We'll upload charts and tables here before sending out dashboard subscriptions.`}
{t`This channel shouldn't really be used by anyone — we'll upload charts and tables here before sending out dashboard subscriptions (it's a Slack requirement).`}
</FormMessage>
<FormField name="slack-files-channel" />
<FormFooter submitTitle={t`Save changes`} />
......
......@@ -24,7 +24,7 @@ const SlackStatusForm = ({ settings }: SlackStatusFormProps): JSX.Element => {
<FormField name="slack-app-token" />
<FormField
name="slack-files-channel"
description={t`We'll upload charts and tables here before sending out dashboard subscriptions.`}
description={t`This channel shouldn't really be used by anyone — we'll upload charts and tables here before sending out dashboard subscriptions (it's a Slack requirement).`}
/>
</Form>
)}
......
......@@ -17,7 +17,7 @@ export const getSlackForm = (
{
name: "slack-files-channel",
type: "input",
title: t`Slack channel name`,
title: t`Public channel to store image files`,
placeholder: "metabase_files",
readOnly,
validate: (value: string) => !value && t`required`,
......
......@@ -2,9 +2,9 @@ import React from "react";
import { fireEvent, render, screen } from "@testing-library/react";
import { ORDERS } from "__support__/sample_database_fixture";
import DataSelectorFieldPicker from "./DataSelectorFieldPicker";
import type { Table } from "metabase-types/api/table";
import type { Field } from "metabase-types/api/field";
import DataSelectorFieldPicker from "./DataSelectorFieldPicker";
const props = {
hasFiltering: true,
......
......@@ -284,7 +284,9 @@ describe("scenarios > admin > settings", () => {
cy.findByText("Metabase on Slack");
cy.findByLabelText("Slack Bot User OAuth Token").type("xoxb");
cy.findByLabelText("Slack channel name").type("metabase_files");
cy.findByLabelText("Public channel to store image files").type(
"metabase_files",
);
cy.button("Save changes").click();
cy.findByText(": invalid token");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment