feat(correction-blocks): Correction box & Loop item + Template

[Takshil-Kunadia]

All Submissions:

• Have you followed the Newspack Contributing guideline?
• Does your code follow the WordPress' coding standards and VIP Go coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Closes https://app.asana.com/0/1209292256643614/1209292256643625

How to test the changes in this Pull Request:

Testing Corrections Box Block:-

1. Create a post add corrections/clarifications.
2. Add Corrections box block to see the corrections.
3. Save the post and view the corrections on the front-end.
4. Modify the corrections on the editor/Add new corrections and click on refresh button to view updated corrections.

Testing Corrections Loop Block Pattern:-

1. Create a new Page and click + > Patterns Corrections Loop.
2. Edit the template to change the title/description.
3. Save & view the page on front-end.

Testing Corrections Archive Block Template:-

1. Go to Site editor > Template > Search "Corrections" > "Corrections Archive".
2. Edit the template to replace Header & Footer / change the title/description.
3. Create a new Page and Select Corrections Archive template.
4. Save & view the page on front-end.

For individual testing of the Corrections Loop item block

1. Add a Query Loop block and add Post Template and Pagination blocks inside it.
2. In Post Template block, Add correction Item block.
3. Set Post type as Corrections in Query Loop block settings. Per page count can also be set here.
4. View the corrections archive.

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

[thomasguillot]

It’s working well! Just a couple of changes:

• The timezone isn’t needed
• The template and pattern strings should be translatable
• This should be restricted to Block Themes only

[Takshil-Kunadia]

Thanks! @leogermani @thomasguillot for taking a look 🙇 .

The timezone isn’t needed

Removed the timezone from heading ✅

The template and pattern strings should be translatable

Wrapped strings in gettext fns. ✅

This should be restricted to Block Themes only

Assuming this is in reference to the Pattern & Template, yes they will only be registered on block themes.

newspack-plugin/includes/corrections/class-corrections.php

Lines 554 to 556 in 45ff164

	public static function register_corrections_block_patterns() {
	if ( ! class_exists( 'WP_Block_Patterns_Registry' ) || ! wp_is_block_theme() ) {
	return;

newspack-plugin/includes/corrections/class-corrections.php

Lines 597 to 599 in 45ff164

	public static function register_corrections_template() {
	if ( ! class_exists( 'WP_Block_Templates_Registry' ) || ! wp_is_block_theme() ) {
	return;

[thomasguillot]

Assuming this is in reference to the Pattern & Template, yes they will only be registered on block themes.

Sorry, I was talking about the Blocks.