Skip to content

Automations paywall#892

Merged
joanagmaia merged 2 commits intomainfrom
bug/automations-paywall
May 22, 2023
Merged

Automations paywall#892
joanagmaia merged 2 commits intomainfrom
bug/automations-paywall

Conversation

@joanagmaia
Copy link
Copy Markdown
Contributor

@joanagmaia joanagmaia commented May 22, 2023

Changes proposed ✍️

  • Show paywall for automations:
    • For Essential users: Show paywall after limit of 2 automations
    • For Growth users: Show paywall after limit of 10 automations

What

🤖 Generated by Copilot at f46bc5c

The code adds feature flag and workflow limit checks for the automation list page, and refreshes the user's feature flags after creating or deleting an automations. This improves the user experience and security of the automation features.

🤖 Generated by Copilot at f46bc5c

We're sailing on the automation sea
With feature-flag and workflow-limit we
Check if we can add or edit with canAddAutomation
And refresh our flags with doRefreshCurrentUser action

Why

How

🤖 Generated by Copilot at f46bc5c

  • Add feature flag and limit logic for automations (link, link, link, link, link, link)
  • Import featureFlag and workflowLimit modules in automation-list-page.vue (link)
  • Define canAddAutomation function to check feature flag and limit before creating or editing automations (link, link)
  • Show dialog if limit is reached or feature flag is disabled (link, link)
  • Import store module in actions.js to access auth actions (link)
  • Dispatch doRefreshCurrentUser action after creating or deleting automations to update feature flags (link, link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@joanagmaia joanagmaia added the Bug Created by Linear-GitHub Sync label May 22, 2023
@joanagmaia joanagmaia requested a review from gaspergrom May 22, 2023 13:35
@joanagmaia joanagmaia self-assigned this May 22, 2023
@joanagmaia joanagmaia merged commit 1eda240 into main May 22, 2023
@joanagmaia joanagmaia deleted the bug/automations-paywall branch May 22, 2023 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants