Skip to content

feat: welcome page #5426

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

Merged
merged 69 commits into from
Jun 4, 2025
Merged

feat: welcome page #5426

merged 69 commits into from
Jun 4, 2025

Conversation

kyle-ssg
Copy link
Member

@kyle-ssg kyle-ssg commented May 6, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Upon creating an organisation or completing the onboarding flow, the user is now directed to a welcome/getting started page.

image

How did you test this code?

  • Signed up via self hosted onboarding -> get to welcome page
  • Created organisation on SaaS -> get to welcome page
  • Tested each todo / link on the getting started page

# Conflicts:
#	frontend/common/utils/utils.tsx
# Conflicts:
#	frontend/web/styles/_variables.scss
# Conflicts:
#	frontend/common/types/requests.ts
#	frontend/common/types/responses.ts
# Conflicts:
#	frontend/common/types/responses.ts
# Conflicts:
#	frontend/common/types/requests.ts
#	frontend/common/types/responses.ts
#	frontend/common/utils/utils.tsx
#	frontend/web/components/pages/HomeAside.tsx
#	frontend/web/components/pages/HomePage.js
#	frontend/web/project/api.js
Copy link
Contributor

@Zaimwa9 Zaimwa9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of comments, as minor I solved them:

  • Double if that could get merged
  • Shrink-0 on resources icons
  • Moved to /onboarding some components
  • Migrated Error/success messages to typescript in messages/XXMessages but only used them in TestWebhook (to test them) and OnboardingForm to not mix-up the scope. I'll create a PR migrating them just after

@github-actions github-actions bot added feature New feature or request and removed feature New feature or request labels Jun 3, 2025
@github-actions github-actions bot added feature New feature or request and removed feature New feature or request labels Jun 3, 2025
@flagsmith flagsmith bot added the flagsmith flag The Issue/PR has a linked feaature flag in Flagsmith label Jun 3, 2025
Copy link

flagsmith bot commented Jun 3, 2025

Flagsmith feature linked: welcome_page
Default Values:

Environment Enabled Value Last Updated (UTC)
Production ❌ Disabled 2025-06-03 10:07:39
Staging ✅ Enabled 2025-05-06 16:51:09
Demo ❌ Disabled 2025-05-06 16:49:55
Self hosted defaults ❌ Disabled 2025-06-03 10:04:58
Demo2 ❌ Disabled 2025-05-06 16:49:55

@kyle-ssg kyle-ssg merged commit 562cf08 into main Jun 4, 2025
27 checks passed
@kyle-ssg kyle-ssg deleted the feat/welcome-page branch June 4, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request flagsmith flag The Issue/PR has a linked feaature flag in Flagsmith front-end Issue related to the React Front End Dashboard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants