Commit for enhancement #7604

Added a checkbox to allow the user to disable the default notifications to the following button actions:
* Save row
* Duplicate row
* Trigger automation
* Execute query
* Delete row
Also adjusted buttonActions.js to reflect the behaviour of that checkbox being ticked.
This commit is contained in:
FlaminWrap 2022-11-13 17:08:23 +00:00
parent e63afd560a
commit 30aa85af0b
6 changed files with 31 additions and 11 deletions

View File

@ -27,6 +27,8 @@
/> />
<Label small /> <Label small />
<Checkbox text="Do not display default notification" bind:value={parameters.notificationOverride} />
<br>
<Checkbox text="Require confirmation" bind:value={parameters.confirm} /> <Checkbox text="Require confirmation" bind:value={parameters.confirm} />
{#if parameters.confirm} {#if parameters.confirm}

View File

@ -95,6 +95,8 @@
/> />
<Label small /> <Label small />
<Checkbox text="Do not display default notification" bind:value={parameters.notificationOverride} />
<br>
<Checkbox text="Require confirmation" bind:value={parameters.confirm} /> <Checkbox text="Require confirmation" bind:value={parameters.confirm} />
{#if parameters.confirm} {#if parameters.confirm}

View File

@ -44,7 +44,8 @@
getOptionLabel={query => query.name} getOptionLabel={query => query.name}
getOptionValue={query => query._id} getOptionValue={query => query._id}
/> />
<Checkbox text="Do not display default notification" bind:value={parameters.notificationOverride} />
<br>
{#if parameters.queryId} {#if parameters.queryId}
<Checkbox text="Require confirmation" bind:value={parameters.confirm} /> <Checkbox text="Require confirmation" bind:value={parameters.confirm} />

View File

@ -95,6 +95,8 @@
/> />
<Label small /> <Label small />
<Checkbox text="Do not display default notification" bind:value={parameters.notificationOverride} />
<br>
<Checkbox text="Require confirmation" bind:value={parameters.confirm} /> <Checkbox text="Require confirmation" bind:value={parameters.confirm} />
{#if parameters.confirm} {#if parameters.confirm}

View File

@ -93,6 +93,8 @@
{/if} {/if}
<Label small /> <Label small />
<Checkbox text="Do not display default notification" bind:value={parameters.notificationOverride} />
<br>
<Checkbox text="Require confirmation" bind:value={parameters.confirm} /> <Checkbox text="Require confirmation" bind:value={parameters.confirm} />
{#if parameters.confirm} {#if parameters.confirm}

View File

@ -17,7 +17,7 @@ import { enrichDataBindings } from "./enrichDataBinding"
import { Helpers } from "@budibase/bbui" import { Helpers } from "@budibase/bbui"
const saveRowHandler = async (action, context) => { const saveRowHandler = async (action, context) => {
const { fields, providerId, tableId } = action.parameters const { fields, providerId, tableId, notificationOverride } = action.parameters
let payload let payload
if (providerId) { if (providerId) {
payload = { ...context[providerId] } payload = { ...context[providerId] }
@ -34,7 +34,10 @@ const saveRowHandler = async (action, context) => {
} }
try { try {
const row = await API.saveRow(payload) const row = await API.saveRow(payload)
notificationStore.actions.success("Row saved")
if (!notificationOverride){
notificationStore.actions.success("Row saved")
}
// Refresh related datasources // Refresh related datasources
await dataSourceStore.actions.invalidateDataSource(row.tableId, { await dataSourceStore.actions.invalidateDataSource(row.tableId, {
@ -49,7 +52,7 @@ const saveRowHandler = async (action, context) => {
} }
const duplicateRowHandler = async (action, context) => { const duplicateRowHandler = async (action, context) => {
const { fields, providerId, tableId } = action.parameters const { fields, providerId, tableId, notificationOverride } = action.parameters
if (providerId) { if (providerId) {
let payload = { ...context[providerId] } let payload = { ...context[providerId] }
if (fields) { if (fields) {
@ -64,7 +67,9 @@ const duplicateRowHandler = async (action, context) => {
delete payload._rev delete payload._rev
try { try {
const row = await API.saveRow(payload) const row = await API.saveRow(payload)
notificationStore.actions.success("Row saved") if (!notificationOverride){
notificationStore.actions.success("Row saved")
}
// Refresh related datasources // Refresh related datasources
await dataSourceStore.actions.invalidateDataSource(row.tableId, { await dataSourceStore.actions.invalidateDataSource(row.tableId, {
@ -80,11 +85,13 @@ const duplicateRowHandler = async (action, context) => {
} }
const deleteRowHandler = async action => { const deleteRowHandler = async action => {
const { tableId, revId, rowId } = action.parameters const { tableId, revId, rowId, notificationOverride } = action.parameters
if (tableId && rowId) { if (tableId && rowId) {
try { try {
await API.deleteRow({ tableId, rowId, revId }) await API.deleteRow({ tableId, rowId, revId })
notificationStore.actions.success("Row deleted") if (!notificationOverride){
notificationStore.actions.success("Row deleted")
}
// Refresh related datasources // Refresh related datasources
await dataSourceStore.actions.invalidateDataSource(tableId, { await dataSourceStore.actions.invalidateDataSource(tableId, {
@ -98,14 +105,16 @@ const deleteRowHandler = async action => {
} }
const triggerAutomationHandler = async action => { const triggerAutomationHandler = async action => {
const { fields } = action.parameters const { fields, notificationOverride } = action.parameters
if (fields) { if (fields) {
try { try {
await API.triggerAutomation({ await API.triggerAutomation({
automationId: action.parameters.automationId, automationId: action.parameters.automationId,
fields, fields,
}) })
notificationStore.actions.success("Automation triggered") if (!notificationOverride){
notificationStore.actions.success("Automation triggered")
}
} catch (error) { } catch (error) {
// Abort next actions // Abort next actions
return false return false
@ -119,7 +128,7 @@ const navigationHandler = action => {
} }
const queryExecutionHandler = async action => { const queryExecutionHandler = async action => {
const { datasourceId, queryId, queryParams } = action.parameters const { datasourceId, queryId, queryParams, notificationOverride } = action.parameters
try { try {
const query = await API.fetchQueryDefinition(queryId) const query = await API.fetchQueryDefinition(queryId)
if (query?.datasourceId == null) { if (query?.datasourceId == null) {
@ -135,7 +144,9 @@ const queryExecutionHandler = async action => {
// Trigger a notification and invalidate the datasource as long as this // Trigger a notification and invalidate the datasource as long as this
// was not a readable query // was not a readable query
if (!query.readable) { if (!query.readable) {
notificationStore.actions.success("Query executed successfully") if (!notificationOverride){
notificationStore.actions.success("Query executed successfully")
}
await dataSourceStore.actions.invalidateDataSource(query.datasourceId) await dataSourceStore.actions.invalidateDataSource(query.datasourceId)
} }