Release/1.2.8

[dkotter]

Changes proposed in this Pull Request:

Version bump and prep for the 1.2.8 release

Closes #456

Steps to test the changes in this Pull Request:

Verify all file changes look accurate

[dkotter]

E2E tests are failing here. Will need to look at those before we proceed

[axi]

switchTab() method search for the last '.wc-tabs > li > a' with name "Availability" but it's now the #bookings_availability that is listed last in the code (and hidden) so test script cannot click it.

<li class="accommodation_bookings_availability_options accommodation_bookings_availability_tab show_if_accommodation-booking accommodation_bookings_tab bookings_availability_tab advanced_options">
    <a href="#accommodation_bookings_availability]"><span>Availability</span></a>
</li>
<li class="bookings_resources_options bookings_resources_tab show_if_booking">
    <a href="#bookings_resources"><span>Resources</span></a>
</li>
<li class="bookings_availability_options bookings_availability_tab show_if_booking">
    <a href="#bookings_availability"><span>Availability</span></a>
</li>

Modifying the selector looks to work : '.wc-tabs > li > a' => '.wc-tabs > li:visible > a'
in tests/e2e/utils/index.js

/**
 * Switch tab in add/edit product page.
 *
 * @param {Page}   page          Playwright page object
 * @param {string} tabName       Tab name
 * @param {string} panelSelector Options Panel selector
 */
export async function switchTab(page, tabName, panelSelector = false) {
	await page
		.locator('.wc-tabs > li:visible > a', { hasText: tabName })
		.last()
		.click();
	if (panelSelector) {
		await expect(page.locator(panelSelector)).toBeVisible();
	}
}

[axi]

E2E tests went through on my machine with the last commit

[iamdharmesh]

Thanks a lot for the suggestion @axi. It worked.

E2E tests went through on my machine with the last commit

Yes, it was working fine for me as well. I have fixed it now. Thanks.

@dkotter we are ready for the next steps here.