3.19 - Preload Fonts - Data insertion part

[piotrbak]

User Story
As an admin, I want to see the preload fonts data in database when manually visit a page

Acceptance Criteria

• The table is created with a fresh install/update
• The following data will be stored in the database:
  • Domain name, ID, device type, status, error message (to be defined)
  • Custom font URLs above the fold (above the fold screensize and threshold are as same as priority elements)
• Any possible error will be saved into the DB and status set to failed
• If no fonts are found, we’ll save this fact to the database and set the status to completed
• Remove failed jobs (the behaviour should be the same as with other priority elements features)

[Khadreal]

Scope a solution

• Create aFonts class in WP_Rocket\Engine\Media\PreloadFonts\Database\Queries, WP_Rocket\Engine\Media\PreloadFonts\Database\Rows, WP_Rocket\Engine\Media\PreloadFonts\Database\Schemas, WP_Rocket\Engine\Media\PreloadFonts\Database\Tables
• The table schema should be as followed:

        id                              -  bigint(20)
        url                             -  varchar(2000)
        is_mobile                 -  boolean (tinyint(1))
        fonts	                 -  longtext
        error_message	-  longtext
        status			-  varchar(255)
        modified		        -  timestamp
        last_accessed	        -  timestamp
        created_at		-  timestamp

• Table should be created if not exists
• Create a new Controller class if not already created within WP_Rocket\Engine\Media\PreloadFonts\AJAX implementing the interface here WP_Rocket\Engine\Common\PerformanceHints\AJAX\ControllerInterface
• Implement the function check_data and add_data
• Add tests

Effort: [S]

[jeawhanlee]

@Khadreal Thanks for the grooming. I believe we still have more areas to cover. We need to add the ajax part that handles the payload from the beacon.

[wordpressfan]

Database structure for the feature is here:
https://www.notion.so/wpmedia/Preload-fonts-database-structure-18bed22a22f080218db6c6a5c963f720

[jeawhanlee]

LGTM

[wordpressfan]

That's merged