Skip to content

Commit

Permalink
changes
Browse files Browse the repository at this point in the history
  • Loading branch information
ramyakrishnai committed Jul 22, 2024
1 parent 2d76baf commit f14f4de
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ import {
setSettings,
initialize as initializeSettings,
} from '../../../utils/api/settings';
import { isEmpty, updateWPSettings } from '../../../utils/api/ecommerce';
import { fetchWPSettings, isEmpty, updateWPSettings } from '../../../utils/api/ecommerce';
import { store as nfdOnboardingStore } from '../../../store';
import { getQueryParam } from '../../../utils';
import { API_REQUEST } from '../../../../constants';
Expand Down Expand Up @@ -341,7 +341,6 @@ const SiteBuild = () => {
};

const trackChapters = () => {
console.count("tracking");
trackInstaWpMigrationEvent();
if ( location.pathname === firstStep.path ) {
trackOnboardingEvent(
Expand Down Expand Up @@ -406,7 +405,7 @@ const SiteBuild = () => {

// Track Migration Initated Event in the Migration Step.
useEffect( () => {
getSettings().then(res => setIsMfeMigrationInitiated(res.nfd_migrate_site))
fetchWPSettings().then( res => setIsMfeMigrationInitiated( res.nfd_migrate_site ) );

Check failure on line 408 in src/OnboardingSPA/components/NewfoldInterfaceSkeleton/SiteBuild/index.js

View workflow job for this annotation

GitHub Actions / Run Lint Checks

Expected parentheses around arrow function argument
trackInstaWpMigrationEvent();
}, [ instaWpMigrationUrl ] );

Check warning on line 410 in src/OnboardingSPA/components/NewfoldInterfaceSkeleton/SiteBuild/index.js

View workflow job for this annotation

GitHub Actions / Run Lint Checks

React Hook useEffect has a missing dependency: 'trackInstaWpMigrationEvent'. Either include it or remove the dependency array

Expand Down

0 comments on commit f14f4de

Please sign in to comment.