Feature Description

When Other is selected as site purpose for tailored metrics, hide Suggested tab in selection panel

Also when switching between other and rest of site purpose answers in KMW admin settings, the confirm change modal should list correct metrics - empty for current or new, depending if we are editing from, or to other answer

Originally raised in this comment

While testing around I spotted a bug when user input is used, selecting other as site purpose will throw console error - complete setup will not execute on last step. This should be included here as well

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

• The suggested group is hidden in the selection panel, when there is no valuable recommendation to show.
• The confirmation metric changes modal does not appear when the user changes the site purpose setting to other.
• The confirmation metric changes modal shows up with currently selected metrics and corresponding recommendations when the user changes the site purpose from other to anything else.
• Selecting the other answer throws no console error during setup.

Implementation Brief

• In assets/js/components/user-input/util/constants.js
  • In USER_INPUT_PURPOSE_TO_CONVERSION_EVENTS_MAPPING add other to the object, with value of empty array
    • This will fix the bug from last point in the AC
• Update assets/js/components/KeyMetrics/ChipTabGroup/index.js
  • Where dynamicGroups is defined, before KEY_METRICS_GROUP_SUGGESTED is added to the array, besides checking for isUserInputCompleted, check also if answerBasedMetrics (which is already defined), is not empty array
• In assets/js/components/KeyMetrics/ConfirmSitePurposeChangeModal.js
  • Where currentMetrics is defined, before getAnswerBasedMetrics is returned, check if savedPurpose?.purpose?.values?.[ 0 ] is other, and return metrics using getKeyMetrics selector from CORE_USER datastore
  • In the existing check

    site-kit-wp/assets/js/components/KeyMetrics/ConfirmSitePurposeChangeModal.js

    Lines 185 to 188 in 2547eae

    	if (
    	savedPurpose?.purpose?.values?.[ 0 ] &&
    	currentMetricsSnapshot === null
    	) {

    include additional AND condition to the list - currentMetrics !== undefined

Test Coverage

• No updates needed

QA Brief

• Setup Site Kit with Analytics module and conversionReporting feature flag enabled
• Go through user input - choose other as site purpose answer. Verify that you can complete setup without errors/being blocked on last step
• Go to Site Kit dashboard, click on change metrics - verify that suggested group is not showing in the tabs when other site answer is saved
• Go to Site Kit Settings > admin settings and edit site purpose from other to any other, it should trigger the modal listing current metrics selection for current metrics
  • Verify that changing the site purpose from any to other, modal is not shown

Changelog entry

• Fix bug that could cause console errors when answering tailored metrics questions.