Skip to content

Conversation

gaspergrom
Copy link
Contributor

@gaspergrom gaspergrom commented Dec 12, 2023

Changes proposed ✍️

What

🤖[deprecated] Generated by Copilot at 5f6fda4

This pull request adds a new banner component to the frontend layout that encourages free and essential plan users to upgrade their workspace. It also adds a new Vuex getter to determine when to show the banner based on the plan and the expiration date.

🤖[deprecated] Generated by Copilot at 5f6fda4

Sing, O Muse, of the cunning coder who devised
A banner bright and bold, to urge the free and frugal
To upgrade their workspace, lest they lose their precious data
And wander in the dark, like the shades of Erebus.

Why

How

🤖[deprecated] Generated by Copilot at 5f6fda4

  • Add a banner component to the main layout template to inform free plan users of the upcoming end of the plan (link, link)
  • Import the Plans module to the tenant module getters to access the plan constants and values (link)
  • Define a new getter showUpgradeEssentialBanner in the tenant module to check the current tenant's plan and date against the limit date of January 1st, 2024 (link)
  • Include the showUpgradeEssentialBanner getter in the showBanner getter logic to conditionally render the banner component in the layout.vue file (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots 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.

@gaspergrom gaspergrom added the Improvement Created by Linear-GitHub Sync label Dec 12, 2023
@gaspergrom gaspergrom self-assigned this Dec 12, 2023
@gaspergrom gaspergrom merged commit 61c0a6f into main Dec 12, 2023
@gaspergrom gaspergrom deleted the improvement/essential-plan-banner branch December 12, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants