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

added function for changing permalink. created performance-hints file… #136

Merged
merged 24 commits into from
Sep 24, 2024

Conversation

hanna-meda
Copy link
Contributor

Added function for changing permalink. created performance-hints files and added steps for 2 scenarios

Description

Partial work on #126

  • added function for changing permalink under utils > page-utils.ts
  • created performance-hints.ts & performance-hints.feature files

This commit covers:
Scenario: Should clear performance hints data when click clear PH in admin bar

Given WPR installed and activated

And performance hints data added to DB

When clear performance hints is clicked in admin bar

Then data is removed from the performance hints tables

Scenario: Should clear performance hints when change permalinks

Given WPR installed and activated

And performance hints data added to DB

When permalinks is changed

Then data is removed from the performance hints tables

Note: Scenarios still need more testing and refinement.

@hanna-meda hanna-meda linked an issue Aug 23, 2024 that may be closed by this pull request
@hanna-meda hanna-meda requested a review from jeawhanlee August 23, 2024 07:47
@jeawhanlee
Copy link
Contributor

@hanna-meda Overall Good Job 🙌

package.json Outdated Show resolved Hide resolved
src/features/performance-hints.feature Outdated Show resolved Hide resolved
src/features/performance-hints.feature Outdated Show resolved Hide resolved
src/features/performance-hints.feature Outdated Show resolved Hide resolved
src/support/steps/general.ts Outdated Show resolved Hide resolved
src/support/steps/performance-hints.ts Outdated Show resolved Hide resolved
@jeawhanlee jeawhanlee requested a review from a team September 19, 2024 14:57
@jeawhanlee jeawhanlee requested a review from Mai-Saad September 20, 2024 10:53
@Mai-Saad
Copy link
Contributor

Mai-Saad commented Sep 20, 2024

test results ✔️
Cucumber.pdf

src/support/steps/performance-hints.ts Outdated Show resolved Hide resolved
@jeawhanlee jeawhanlee merged commit 221c9d2 into develop Sep 24, 2024
1 of 2 checks passed
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.

Clear lcp/performance hints data tests
4 participants