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.

Sign up for GitHub

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

Jump to bottom

Remove the client-side performance module #481

Closed
radeklawgmin opened this issue Feb 5, 2019 · 2 comments
Closed

Remove the client-side performance module #481

radeklawgmin opened this issue Feb 5, 2019 · 2 comments
Labels
enhancement

Comments

@radeklawgmin
Copy link
Contributor

radeklawgmin commented Feb 5, 2019 •
edited
Loading

If you look at the current list of AET features threre is the client-side performance module that allegedly allows to run client-side performance tests with AET. It makes use of ySlow that is quite old and looks like no longer supported nor compatible with browsers.
Let's remove this module from the code and documentation to avoid potential issues and misunderstanding.
@radeklawgmin
Copy link
Contributor Author

check these urls:

@radeklawgmin radeklawgmin changed the title Remove the Accessibilty module Remove the client-side performance module Feb 5, 2019
@radeklawgmin
Copy link
Contributor Author

Note that there is a dedicated issue for implementing client-side performance testing module based on Lighthouse at #368.

@radeklawgmin radeklawgmin mentioned this issue Feb 25, 2019
Merged
5 tasks
@KarolFrankiewicz KarolFrankiewicz self-assigned this Oct 14, 2019
@KarolFrankiewicz KarolFrankiewicz mentioned this issue Oct 16, 2019
Merged
5 tasks
@tMaxx tMaxx closed this as completed Sep 7, 2020
@KarolFrankiewicz KarolFrankiewicz removed their assignment Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@KarolFrankiewicz @tMaxx @radeklawgmin