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

Add federicoleva.eu #853

Merged
merged 1 commit into from Jun 2, 2022
Merged

Add federicoleva.eu #853

merged 1 commit into from Jun 2, 2022

Conversation

nemobis
Copy link
Contributor

@nemobis nemobis commented May 29, 2022

This PR is:

  • Adding a new domain
  • Updating existing domain size
  • Changing domain name
  • Removing existing domain from list
  • Website code changes (512kb.club site)
  • Other not listed
  • [?] I used the uncompressed size of the site
  • I have included a link to the GTMetrix report
  • The domain is in the correct alphabetical order
  • This site is not a ultra lightweight site
  • The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

  • Check to confirm
- domain: federicoleva.eu
  url: https://federicoleva.eu/
  size: 273
  last_checked: 2022-05-29

GTMetrix Report: https://gtmetrix.com/reports/federicoleva.eu/ehkx9lte/

WebPageTest: https://www.webpagetest.org/result/220529_BiDcV4_438/1/details/

Copy link
Collaborator

@garritfra garritfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like you used the compressed size of the page. Please update the value to be the uncompressed size, as noted in the FAQ. Thanks!🙂

@nemobis
Copy link
Contributor Author

nemobis commented Jun 1, 2022

Thanks for looking. I don't see where to get that figure from gtmetrix, but I calculated it myself now. Sorry for missing that bit in the instructions, I added the figure from the waterfall tab now.

@garritfra
Copy link
Collaborator

Thanks! Looks good to me now. :)

@garritfra garritfra merged commit ec9eb6e into kevquirk:master Jun 2, 2022
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.

None yet

2 participants