Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quickstart v2 #1832

Merged
merged 13 commits into from
Nov 14, 2023
Merged

Quickstart v2 #1832

merged 13 commits into from
Nov 14, 2023

Conversation

gaspergrom
Copy link
Contributor

@gaspergrom gaspergrom commented Nov 9, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 24a166f

This pull request implements the quickstart guide v2 feature, which provides a new set of guides and UI for the users to learn how to use the app. It adds, updates, or deletes several files in the backend and frontend modules to handle the feature flag, the new guide types, the tenant settings, the database migrations, and the menu and dashboard components. It also refactors some existing code to improve consistency and readability.

🤖 Generated by Copilot at 24a166f

Oh, we're the crew of the quickstart module
We build the guides for the new users' trouble
We heave and we haul on the cr-menu-quickstart
And we view the organizations with the tenant service

Why

How

🤖 Generated by Copilot at 24a166f

  • Add a new API endpoint and service function for the tenant service to handle the view organizations request, which updates the tenant settings to mark the organizations as viewed (link, link)
  • Add a call to the tenant service function in the organization list page component, which is executed when the user visits the organization list page (link)
  • Add and modify migration files to drop and add back the organizationsViewed column in the settings table in the database, depending on the feature flag for the quickstart guide v2 (link, link)
  • Add the organizationsViewed attribute to the settings model in the database (link)
  • Add and modify the quickstart guide types to include the new guide types, the learnMoreLink property, and the default guides for the quickstart guide v2 feature (link, link, link, link, link, link, link, link)
  • Modify the quickstart guide service to use the default guides for the quickstart guide v2 feature if the feature flag is enabled, and to add the logic for the new guide types and the organizationsViewed flag (link, link, link, link, link)
  • Add a new module folder for the quickstart guide v2 feature, which contains the components, store, and types for the feature, and register the module in the modules index file (link, link, link, link, link)
  • Add a new menu item component for the quickstart guide v2 feature, which renders the menu item with a welcome message, an emoji, and a link to the quickstart page, and add the component to the menu component with a condition to check the feature flag (link, link, link, link)
  • Add a condition to the dashboard guides component to only render it if the feature flag for the quickstart guide v2 is not enabled, and modify the imports and the store variable to reflect the renaming of the module folder (link, link, link, link, link, link, link)
  • Delete the files for the quickstart guide store that are no longer needed (link, link, 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 Nov 9, 2023
@gaspergrom gaspergrom self-assigned this Nov 9, 2023
@gaspergrom gaspergrom merged commit fa714f3 into main Nov 14, 2023
9 checks passed
@gaspergrom gaspergrom deleted the improvement/quickstart branch November 14, 2023 09:56
gaspergrom added a commit that referenced this pull request Dec 7, 2023
(cherry picked from commit fa714f3)
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.

None yet

2 participants