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

PRESS2-1267-update-fork-new #363

Merged
merged 19 commits into from
Dec 4, 2023

Conversation

girish-lokapure
Copy link
Contributor

No description provided.

@girish-lokapure girish-lokapure changed the title initial commit PRESS2-1267-update-fork-new Nov 14, 2023
@officiallygod officiallygod added the Code Review The PR is in Code Review label Nov 18, 2023
Copy link
Member

@officiallygod officiallygod left a comment

Choose a reason for hiding this comment

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

Please do merge the latest enhance/ai-onboarding branch as I think there are numerous changes.

src/OnboardingSPA/steps/Complete/contents.js Outdated Show resolved Hide resolved
src/OnboardingSPA/steps/TheFork/contents.js Outdated Show resolved Hide resolved
@@ -14,9 +14,19 @@ import { forwardRef, useEffect } from '@wordpress/element';
import { __unstableUseNavigateRegions as useNavigateRegions } from '@wordpress/components';
import { __ } from '@wordpress/i18n';
import { useMergeRefs } from '@wordpress/compose';
import bgAiImg from '../../static/images/ai_bg.jpg';
Copy link
Member

Choose a reason for hiding this comment

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

Can't we use the same way of mentioning in _icons and not directly reference a static file?

src/OnboardingSPA/components/StartOptions/index.js Outdated Show resolved Hide resolved
src/OnboardingSPA/components/StartOptions/index.js Outdated Show resolved Hide resolved
src/OnboardingSPA/components/StartOptions/index.js Outdated Show resolved Hide resolved
@girish-lokapure girish-lokapure force-pushed the PRESS2-1267-update-fork-new branch from 6bd3d12 to 813787f Compare November 28, 2023 16:38
@girish-lokapure girish-lokapure merged commit e7e6127 into enhance/ai-onboarding Dec 4, 2023
1 check passed
@diwanshuster diwanshuster deleted the PRESS2-1267-update-fork-new branch March 6, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Review The PR is in Code Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants