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

feat(Illustrations): Add 3 new onboarding quiz illustrations #2758

Merged
merged 2 commits into from
Jul 20, 2023

Conversation

zippyzow
Copy link
Contributor

@zippyzow zippyzow commented Jul 19, 2023

Overview

Add 3 new illustrations for our revamp of the onboarding quiz.

PR Checklist

Testing Instructions

Storybook: Under Illustrations, see Goal Purpose, Interest, and Knowledge.

Mono PR Env: As a logged in user navigate to the onboarding quiz /welcome/find-a-course. See appropriate image and text next to each question. Details of implementation/axe will be addressed in separate mono ticket. This is just to make sure the images are being imported from Gamut ok.

PR Links and Envs

Repository PR Link PR Env
Monolith Monolith PR
Portal Portal PR Portal Env

@zippyzow zippyzow marked this pull request as ready for review July 20, 2023 16:29
@zippyzow zippyzow requested review from a team, saghdaey and cstatro and removed request for a team July 20, 2023 16:29

import { IllustrationProps } from './types';

export const Goal: React.FC<IllustrationProps> = ({
Copy link
Contributor

Choose a reason for hiding this comment

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

non-blocking and just a question, should we call this one purpose instead of goal so it matches the id of that question?
the other two are a little closer tied together
Screenshot 2023-07-20 at 1 24 45 PM

@saghdaey saghdaey self-requested a review July 20, 2023 17:25
Copy link
Contributor

@saghdaey saghdaey left a comment

Choose a reason for hiding this comment

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

LGTM!! so nice to see the quiz coming together (in the monorepo pr env)

@codecademydev
Copy link
Collaborator

📬Published Alpha Packages:

@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]

@codecademydev
Copy link
Collaborator

🚀 Styleguide deploy preview ready!

https://64b971411614c10fb994e997--gamut-preview.netlify.app

Deploy Logs

@zippyzow zippyzow added the Ship It :shipit: Automerge this PR when possible label Jul 20, 2023
@codecademydev codecademydev merged commit 07a2ad7 into main Jul 20, 2023
14 of 15 checks passed
@codecademydev codecademydev deleted the hr-disc-1218-illustrations branch July 20, 2023 17:49
@codecademydev codecademydev removed the Ship It :shipit: Automerge this PR when possible label Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants