[Feature]: Preparation for WordPress org release by theMasudRana · Pull Request #57 · rtCamp/carousel-kit

theMasudRana · 2026-02-24T09:04:17Z

Description: Prepare the plugin for WordPress.org release

Issue: https://github.com/rtCamp/support/issues/462#issuecomment-3982065178

…s.org distribution

… and bugs metadata

…or plugin structure

…ibution

… feat/prepair-for-wp-org

…t.php for direct access protection

Copilot

Pull request overview

Prepares the Carousel Kit plugin for a WordPress.org release by adding WP.org-required metadata/artifacts, localizing assets, and hardening direct access.

Changes:

Adds WordPress.org packaging assets (readme.txt, .distignore) and localization template (POT) plus an npm script to generate it.
Replaces externally-hosted placeholder images in example patterns with bundled local assets.
Adds direct-access prevention and an index.php “silence is golden” file.

Reviewed changes

Copilot reviewed 10 out of 17 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
readme.txt	Adds WordPress.org readme metadata, description, screenshots, changelog.
package.json	Adds repository/bugs metadata and an i18n POT generation script.
languages/carousel-kit.pot	Adds translation template for strings.
index.php	Adds “silence is golden” index to prevent directory listing.
examples/patterns/logo-showcase.php	Switches pattern images from external placeholders to bundled SVGs.
examples/patterns/hero-carousel.php	Switches hero slides from external placeholders to bundled WEBPs.
examples/data/images/logo-placeholder-*.svg	Adds bundled placeholder logo SVG assets.
composer.json	Removes a production dependency from `require`.
composer.lock	Updates lockfile to reflect removed production dependency.
carousel-kit.php	Updates license header and adds direct access guard.
Makefile	Adjusts dist exclusions (README.md no longer excluded).
.distignore	Adds ignore rules for WordPress.org distribution packaging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-03T09:34:48Z

readme.txt

+== Changelog ==
+
+= 1.0.2 =
+* Fix: Demo pattern now shows 4 slides per view
+* Fix: Replace PNG images with optimized WEBP format
+* New: Setup wizard styles
+* New: Slide appender and setup wizard
+


The readme changelog/upgrade notice stops at 1.0.2, but the plugin version is 1.0.3. Add a 1.0.3 entry (or update the current entries) so WordPress.org users can see what changed in the version being released.

cc @theMasudRana

Changelog added for version 1.0.3

readme.txt

package.json

Copilot · 2026-03-03T09:34:49Z

package.json

+	"repository": {
+		"type": "git",
+		"url": "https://github.com/rtCamp/carousel-kit"
+	},
+	"bugs": {
+		"url": "https://github.com/rtCamp/carousel-kit/issues"
+	},


This hunk introduces tab-indented properties (repository, bugs) while the rest of package.json uses spaces. Please normalize indentation so automated formatters/linters and future diffs remain consistent.

Suggested change

"repository": {

"type": "git",

"url": "https://github.com/rtCamp/carousel-kit"

},

"bugs": {

"url": "https://github.com/rtCamp/carousel-kit/issues"

},

"repository": {

"type": "git",

"url": "https://github.com/rtCamp/carousel-kit"

},

"bugs": {

"url": "https://github.com/rtCamp/carousel-kit/issues"

},

@theMasudRana

Use consistent formatting, stick with either spaces or tabs, not both in a single file.

In the .editorconfig of the plugin, its specified that 2 spaces are to be used for indentation of json files, so stick with that or update the .editorconfig and update all json files to follow that.

Formatted the file with 2 spaces as per .editorconfig

Copilot · 2026-03-03T09:34:49Z

composer.json

 	"require": {
-		"php": "^8.2",
-		"scrivo/highlight.php": "^9.18"
+		"php": "^8.2"
 	},


composer.json requires PHP ^8.2, but the plugin header (carousel-kit.php) and readme.txt both declare Requires PHP: 7.4. For WordPress.org, these should be aligned with the actual runtime requirement; either lower composer.json's PHP constraint/platform or bump the plugin/readme requirement to 8.2 if that's the real minimum.

@theMasudRana

Have we tested this plugin with PHP 7.4?
If not, then whichever is the lowest version of PHP you've tested this plugin with, that should be the required PHP version.

Copilot · 2026-03-03T09:34:50Z

readme.txt

+Requires at least: 6.5
+Tested up to: 6.9
+Requires PHP: 7.4
+Stable tag: 1.0.2


Stable tag is set to 1.0.2, but the plugin header and package.json both declare version 1.0.3 (and the POT header also says 1.0.3). For WordPress.org releases, the readme stable tag should match the released plugin version (or be trunk).

Suggested change

Stable tag: 1.0.2

Stable tag: 1.0.3

cc @theMasudRana

Updated the stable tag.

up1512001

@theMasudRana Added few suggestions.

up1512001 · 2026-03-03T10:10:04Z

carousel-kit.php

@@ -8,17 +8,21 @@
 * Author:      rtCamp
 * Author URI:  https://rtcamp.com
 * Contributors: iamdanih17, immasud


please add rtcamp as well

up1512001 · 2026-03-03T10:11:35Z

carousel-kit.php

+	exit; // Exit if accessed directly.
+}
+
 define( 'CAROUSEL_KIT_PATH', untrailingslashit( plugin_dir_path( __FILE__ ) ) );


if possible wrap this all const into function and add namespace to this file.
reference: https://github.com/rtCamp/OneUpdate/blob/e5920c452fffc70a2d7d922a2c80d44ff065a216/oneupdate.php#L29

up1512001 · 2026-03-03T10:13:45Z

carousel-kit.php

 define( 'CAROUSEL_KIT_BUILD_PATH', CAROUSEL_KIT_PATH . '/build' );
 define( 'CAROUSEL_KIT_BUILD_URL', CAROUSEL_KIT_URL . '/build' );

 require_once CAROUSEL_KIT_PATH . '/vendor/autoload.php';


add file exist check if someone downloads from GH then activating this plugin should give admin notice instead of fatal error.

https://github.com/rtCamp/OneAccess/blob/b8ac1e297c90949e98c2a1ad696380c8b033a15c/inc/Autoloader.php#L61

up1512001 · 2026-03-03T10:15:21Z

package.json

@@ -1,3 +1,3 @@
 {
  "name": "carousel-kit",
  "version": "1.0.3",


I suggest to remove version from composer & package file to avoid maintaining it on multiple places.

up1512001 · 2026-03-03T10:18:34Z

readme.txt

+3. Query Loop integration for dynamic content
+4. Frontend carousel with autoplay
+
+== Changelog ==


in plugin root we should have CHANGELOG.md https://github.com/rtCamp/godam/blob/main/CHANGELOG.md

and add just reference of that be adding at latest 2 release info

https://github.com/rtCamp/godam/blob/2456c02e667587571eb8be45f3df8f9d55e449f3/readme.txt#L247C1-L300C83

up1512001 · 2026-03-03T10:19:22Z

readme.txt

+* Query Loop integration
+* RTL and accessibility support
+
+== Upgrade Notice ==


upgrade notice should be for latest plugin version and if its doing any breaking changes.

https://wordpress.org/plugins/readme.txt#:~:text=%3D%3D%20Upgrade%20Notice%20%3D%3D

https://wp-kama.com/handbook/plugin/wordpress-org/readme-txt#upgrade-notice

…d changelog entries for fixes

theMasudRana added 6 commits February 24, 2026 15:01

chore: remove unused dependency scrivo/highlight.php from composer files

deaef43

feat: add .distignore file to exclude unnecessary files from WordPres…

2760527

…s.org distribution

feat: add localization support and update package.json for repository…

2268cb8

… and bugs metadata

feat: add readme.txt with plugin details and installation instructions

e8cdd4a

feat: update license and version in carousel-kit.php; add index.php f…

4e5fe9f

…or plugin structure

feat: add screenshot for plugin preview in WordPress.org

c3826fc

theMasudRana self-assigned this Feb 24, 2026

theMasudRana added 2 commits February 24, 2026 19:03

feat: update .distignore to refine exclusions for WordPress.org distr…

c4e35cd

…ibution

Merge branch 'develop' of https://github.com/rtCamp/carousel-kit into…

11ae3a9

… feat/prepair-for-wp-org

theMasudRana requested review from danish17 and removed request for danish17 February 26, 2026 13:53

theMasudRana added 4 commits March 2, 2026 19:50

Merge branch 'develop' of https://github.com/rtCamp/carousel-kit into…

712f3b4

… feat/prepair-for-wp-org

feat: add placeholder logos and update hero carousel pattern

bdd4b52

feat: update .distignore to exclude README.md and enhance carousel-ki…

e5ee596

…t.php for direct access protection

feat: Updated pot file

cd5e061

theMasudRana marked this pull request as ready for review March 2, 2026 14:32

theMasudRana requested review from danish17, gagan0123 and up1512001 March 2, 2026 14:35

up1512001 requested a review from Copilot March 3, 2026 09:30

Copilot started reviewing on behalf of up1512001 March 3, 2026 09:30 View session

Copilot AI reviewed Mar 3, 2026

View reviewed changes

up1512001 reviewed Mar 3, 2026

View reviewed changes

theMasudRana added 2 commits March 4, 2026 01:29

feat: update PHP requirement to 8.2, bump stable tag to 1.0.3, and ad…

e32acaf

…d changelog entries for fixes

chore: format package.json for consistency in spacing

acd5906

Conversation

theMasudRana commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

up1512001 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

theMasudRana commented Feb 24, 2026 •

edited

Loading