Fixed Text Domain in attributes.php

[DAnn2012]

Fixed Text Domain in attributes.php

Summary by CodeRabbit

Release Notes

This update contains internal maintenance improvements with no visible changes to end-user functionality. The blog posts component has been updated to use standardized configuration for localization handling, ensuring consistent translation practices across the application.

• Chores
  • Updated internal localization configuration for blog posts component

[coderabbitai]

Walkthrough

Translation domain constants were standardized in the deprecated v2 blog posts block. Two attribute defaults (nextLabel and previousLabel) were updated to use the STACKABLE_I18N constant instead of hardcoded 'stackable' string for the translation domain.

Changes

Cohort / File(s)	Change Summary
Translation domain standardization src/deprecated/v2/block/blog-posts/attributes.php	Updated two attribute defaults to use STACKABLE_I18N constant for translation domain instead of hardcoded 'stackable' string

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Straightforward constant substitution with no logic changes
• Limited scope to a single file with repetitive pattern application

Poem

🐰 A constant blooms where strings once grew,
Translation domains, now tried and true,
No more hardcoding, neat and clean—
The cleanest refactor ever seen!
hops away satisfied ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "Fixed Text Domain in attributes.php" is directly aligned with the changeset. The modifications specifically involve updating the translation domain from the hardcoded string 'stackable' to the STACKABLE_I18N constant in the attributes.php file for two attribute defaults. The title is concise, clear, and specific—it accurately identifies both the file and the nature of the fix. A teammate scanning the commit history would readily understand that this addresses a text domain localization issue.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cfefc88 and 481a57c.

📒 Files selected for processing (1)

• src/deprecated/v2/block/blog-posts/attributes.php (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)

• GitHub Check: PHP 8.2 and WP latest
• GitHub Check: PHP 8.2 and WP 6.6.2
• GitHub Check: PHP 8.2 and WP 6.7.2
• GitHub Check: PHP 8.2 and WP 6.5.5
• GitHub Check: PHP 7.3 and WP 6.5.5
• GitHub Check: PHP 7.3 and WP latest
• GitHub Check: build

🔇 Additional comments (1)

src/deprecated/v2/block/blog-posts/attributes.php (1)

1845-1845: Text domain standardization verified and approved.

The STACKABLE_I18N constant is properly defined in plugin.php:52 with value 'stackable-ultimate-gutenberg-blocks'. The replacement of hardcoded 'stackable' with this constant is already consistently applied throughout the codebase, including the deprecated v2 blog-posts block. No hardcoded text domains remain in translation function calls. Changes follow WordPress best practices and pose no risk.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[DAnn2012]

Hello @bfintal

When you can, could you take a look at this PR of mine?

Thanks.