Skip to content

Commit e9aede0

Browse files
waleedlatif1waleedlatif1
authored
improvement(triggers): moved save configuration above instructions for better visibility, fixed styling inconsistencies (#2699)
1 parent ba2377f commit e9aede0

File tree

85 files changed

+817
-822
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

85 files changed

+817
-822
lines changed

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/trigger-save/trigger-save.tsx‎

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -361,9 +361,9 @@ export function TriggerSave({
361361
onClick={handleSave}
362362
disabled={disabled || isProcessing}
363363
className={cn(
364-
'h-[32px] flex-1 rounded-[8px] px-[12px] transition-all duration-200',
365-
saveStatus === 'saved' && 'bg-green-600 hover:bg-green-700',
366-
saveStatus === 'error' && 'bg-red-600 hover:bg-red-700'
364+
'flex-1',
365+
saveStatus === 'saved' && '!bg-green-600 !text-white hover:!bg-green-700',
366+
saveStatus === 'error' && '!bg-red-600 !text-white hover:!bg-red-700'
367367
)}
368368
>
369369
{saveStatus === 'saving' && 'Saving...'}
@@ -373,12 +373,7 @@ export function TriggerSave({
373373
</Button>
374374

375375
{webhookId && (
376-
<Button
377-
variant='default'
378-
onClick={handleDeleteClick}
379-
disabled={disabled || isProcessing}
380-
className='h-[32px] rounded-[8px] px-[12px]'
381-
>
376+
<Button variant='default' onClick={handleDeleteClick} disabled={disabled || isProcessing}>
382377
<Trash className='h-[14px] w-[14px]' />
383378
</Button>
384379
)}

‎apps/sim/triggers/airtable/webhook.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,14 @@ export const airtableWebhookTrigger: TriggerConfig = {
4747
defaultValue: false,
4848
mode: 'trigger',
4949
},
50+
{
51+
id: 'triggerSave',
52+
title: '',
53+
type: 'trigger-save',
54+
hideFromPreview: true,
55+
mode: 'trigger',
56+
triggerId: 'airtable_webhook',
57+
},
5058
{
5159
id: 'triggerInstructions',
5260
title: 'Setup Instructions',
@@ -67,14 +75,6 @@ export const airtableWebhookTrigger: TriggerConfig = {
6775
.join(''),
6876
mode: 'trigger',
6977
},
70-
{
71-
id: 'triggerSave',
72-
title: '',
73-
type: 'trigger-save',
74-
hideFromPreview: true,
75-
mode: 'trigger',
76-
triggerId: 'airtable_webhook',
77-
},
7878
],
7979

8080
outputs: {

‎apps/sim/triggers/calendly/invitee_canceled.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,18 @@ export const calendlyInviteeCanceledTrigger: TriggerConfig = {
3838
value: 'calendly_invitee_canceled',
3939
},
4040
},
41+
{
42+
id: 'triggerSave',
43+
title: '',
44+
type: 'trigger-save',
45+
hideFromPreview: true,
46+
mode: 'trigger',
47+
triggerId: 'calendly_invitee_canceled',
48+
condition: {
49+
field: 'selectedTriggerId',
50+
value: 'calendly_invitee_canceled',
51+
},
52+
},
4153
{
4254
id: 'triggerInstructions',
4355
title: 'Setup Instructions',
@@ -61,18 +73,6 @@ export const calendlyInviteeCanceledTrigger: TriggerConfig = {
6173
value: 'calendly_invitee_canceled',
6274
},
6375
},
64-
{
65-
id: 'triggerSave',
66-
title: '',
67-
type: 'trigger-save',
68-
hideFromPreview: true,
69-
mode: 'trigger',
70-
triggerId: 'calendly_invitee_canceled',
71-
condition: {
72-
field: 'selectedTriggerId',
73-
value: 'calendly_invitee_canceled',
74-
},
75-
},
7676
],
7777

7878
outputs: buildInviteeOutputs(),

‎apps/sim/triggers/calendly/invitee_created.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,18 @@ export const calendlyInviteeCreatedTrigger: TriggerConfig = {
4747
value: 'calendly_invitee_created',
4848
},
4949
},
50+
{
51+
id: 'triggerSave',
52+
title: '',
53+
type: 'trigger-save',
54+
hideFromPreview: true,
55+
mode: 'trigger',
56+
triggerId: 'calendly_invitee_created',
57+
condition: {
58+
field: 'selectedTriggerId',
59+
value: 'calendly_invitee_created',
60+
},
61+
},
5062
{
5163
id: 'triggerInstructions',
5264
title: 'Setup Instructions',
@@ -70,18 +82,6 @@ export const calendlyInviteeCreatedTrigger: TriggerConfig = {
7082
value: 'calendly_invitee_created',
7183
},
7284
},
73-
{
74-
id: 'triggerSave',
75-
title: '',
76-
type: 'trigger-save',
77-
hideFromPreview: true,
78-
mode: 'trigger',
79-
triggerId: 'calendly_invitee_created',
80-
condition: {
81-
field: 'selectedTriggerId',
82-
value: 'calendly_invitee_created',
83-
},
84-
},
8585
],
8686

8787
outputs: buildInviteeOutputs(),

‎apps/sim/triggers/calendly/routing_form_submitted.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,18 @@ export const calendlyRoutingFormSubmittedTrigger: TriggerConfig = {
3838
value: 'calendly_routing_form_submitted',
3939
},
4040
},
41+
{
42+
id: 'triggerSave',
43+
title: '',
44+
type: 'trigger-save',
45+
hideFromPreview: true,
46+
mode: 'trigger',
47+
triggerId: 'calendly_routing_form_submitted',
48+
condition: {
49+
field: 'selectedTriggerId',
50+
value: 'calendly_routing_form_submitted',
51+
},
52+
},
4153
{
4254
id: 'triggerInstructions',
4355
title: 'Setup Instructions',
@@ -61,18 +73,6 @@ export const calendlyRoutingFormSubmittedTrigger: TriggerConfig = {
6173
value: 'calendly_routing_form_submitted',
6274
},
6375
},
64-
{
65-
id: 'triggerSave',
66-
title: '',
67-
type: 'trigger-save',
68-
hideFromPreview: true,
69-
mode: 'trigger',
70-
triggerId: 'calendly_routing_form_submitted',
71-
condition: {
72-
field: 'selectedTriggerId',
73-
value: 'calendly_routing_form_submitted',
74-
},
75-
},
7676
],
7777

7878
outputs: buildRoutingFormOutputs(),

‎apps/sim/triggers/calendly/webhook.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,18 @@ export const calendlyWebhookTrigger: TriggerConfig = {
3737
value: 'calendly_webhook',
3838
},
3939
},
40+
{
41+
id: 'triggerSave',
42+
title: '',
43+
type: 'trigger-save',
44+
hideFromPreview: true,
45+
mode: 'trigger',
46+
triggerId: 'calendly_webhook',
47+
condition: {
48+
field: 'selectedTriggerId',
49+
value: 'calendly_webhook',
50+
},
51+
},
4052
{
4153
id: 'triggerInstructions',
4254
title: 'Setup Instructions',
@@ -60,18 +72,6 @@ export const calendlyWebhookTrigger: TriggerConfig = {
6072
value: 'calendly_webhook',
6173
},
6274
},
63-
{
64-
id: 'triggerSave',
65-
title: '',
66-
type: 'trigger-save',
67-
hideFromPreview: true,
68-
mode: 'trigger',
69-
triggerId: 'calendly_webhook',
70-
condition: {
71-
field: 'selectedTriggerId',
72-
value: 'calendly_webhook',
73-
},
74-
},
7575
],
7676

7777
outputs: {

‎apps/sim/triggers/circleback/meeting_completed.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,24 +40,24 @@ export const circlebackMeetingCompletedTrigger: TriggerConfig = {
4040
},
4141
},
4242
{
43-
id: 'triggerInstructions',
44-
title: 'Setup Instructions',
43+
id: 'triggerSave',
44+
title: '',
45+
type: 'trigger-save',
4546
hideFromPreview: true,
46-
type: 'text',
47-
defaultValue: circlebackSetupInstructions('All meeting data'),
4847
mode: 'trigger',
48+
triggerId: 'circleback_meeting_completed',
4949
condition: {
5050
field: 'selectedTriggerId',
5151
value: 'circleback_meeting_completed',
5252
},
5353
},
5454
{
55-
id: 'triggerSave',
56-
title: '',
57-
type: 'trigger-save',
55+
id: 'triggerInstructions',
56+
title: 'Setup Instructions',
5857
hideFromPreview: true,
58+
type: 'text',
59+
defaultValue: circlebackSetupInstructions('All meeting data'),
5960
mode: 'trigger',
60-
triggerId: 'circleback_meeting_completed',
6161
condition: {
6262
field: 'selectedTriggerId',
6363
value: 'circleback_meeting_completed',

‎apps/sim/triggers/circleback/meeting_notes.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,24 +40,24 @@ export const circlebackMeetingNotesTrigger: TriggerConfig = {
4040
},
4141
},
4242
{
43-
id: 'triggerInstructions',
44-
title: 'Setup Instructions',
43+
id: 'triggerSave',
44+
title: '',
45+
type: 'trigger-save',
4546
hideFromPreview: true,
46-
type: 'text',
47-
defaultValue: circlebackSetupInstructions('Meeting notes and action items'),
4847
mode: 'trigger',
48+
triggerId: 'circleback_meeting_notes',
4949
condition: {
5050
field: 'selectedTriggerId',
5151
value: 'circleback_meeting_notes',
5252
},
5353
},
5454
{
55-
id: 'triggerSave',
56-
title: '',
57-
type: 'trigger-save',
55+
id: 'triggerInstructions',
56+
title: 'Setup Instructions',
5857
hideFromPreview: true,
58+
type: 'text',
59+
defaultValue: circlebackSetupInstructions('Meeting notes and action items'),
5960
mode: 'trigger',
60-
triggerId: 'circleback_meeting_notes',
6161
condition: {
6262
field: 'selectedTriggerId',
6363
value: 'circleback_meeting_notes',

‎apps/sim/triggers/circleback/webhook.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -49,24 +49,24 @@ export const circlebackWebhookTrigger: TriggerConfig = {
4949
},
5050
},
5151
{
52-
id: 'triggerInstructions',
53-
title: 'Setup Instructions',
52+
id: 'triggerSave',
53+
title: '',
54+
type: 'trigger-save',
5455
hideFromPreview: true,
55-
type: 'text',
56-
defaultValue: circlebackSetupInstructions('All events'),
5756
mode: 'trigger',
57+
triggerId: 'circleback_webhook',
5858
condition: {
5959
field: 'selectedTriggerId',
6060
value: 'circleback_webhook',
6161
},
6262
},
6363
{
64-
id: 'triggerSave',
65-
title: '',
66-
type: 'trigger-save',
64+
id: 'triggerInstructions',
65+
title: 'Setup Instructions',
6766
hideFromPreview: true,
67+
type: 'text',
68+
defaultValue: circlebackSetupInstructions('All events'),
6869
mode: 'trigger',
69-
triggerId: 'circleback_webhook',
7070
condition: {
7171
field: 'selectedTriggerId',
7272
value: 'circleback_webhook',

‎apps/sim/triggers/generic/webhook.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,14 @@ export const genericWebhookTrigger: TriggerConfig = {
5656
'Define the expected JSON input schema for this webhook (optional). Use type "files" for file uploads.',
5757
mode: 'trigger',
5858
},
59+
{
60+
id: 'triggerSave',
61+
title: '',
62+
type: 'trigger-save',
63+
hideFromPreview: true,
64+
mode: 'trigger',
65+
triggerId: 'generic_webhook',
66+
},
5967
{
6068
id: 'triggerInstructions',
6169
title: 'Setup Instructions',
@@ -76,14 +84,6 @@ export const genericWebhookTrigger: TriggerConfig = {
7684
.join(''),
7785
mode: 'trigger',
7886
},
79-
{
80-
id: 'triggerSave',
81-
title: '',
82-
type: 'trigger-save',
83-
hideFromPreview: true,
84-
mode: 'trigger',
85-
triggerId: 'generic_webhook',
86-
},
8787
],
8888

8989
outputs: {},

0 commit comments

Comments
 (0)