Skip to content

Feature: PR integration#7

Merged
up1512001 merged 18 commits intomainfrom
feature/pull-requests
Sep 19, 2025
Merged

Feature: PR integration#7
up1512001 merged 18 commits intomainfrom
feature/pull-requests

Conversation

@up1512001
Copy link
Copy Markdown
Member

@up1512001 up1512001 commented Sep 8, 2025

  • Added an admin page called Pull Requests where all connected sites' PRs will be visible.
  • Users can search PRs based on keywords or PR numbers, and also filter PRs by repositories or status.
  • On a single PR, there will be three actions:
    • View details
  • Removed the cron job for weekly S3 history cleanup table.
  • Added utils/constants classes for code cleanup.
  • fix security issue reported by CodeQL

@up1512001 up1512001 self-assigned this Sep 8, 2025
@up1512001 up1512001 marked this pull request as ready for review September 15, 2025 11:13
Copilot AI review requested due to automatic review settings September 15, 2025 11:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a pull requests feature that allows users to view and manage GitHub pull requests from connected sites within the OneUpdate admin interface. The feature includes search and filtering capabilities and removes the weekly S3 history cleanup cron job.

Key changes include:

  • Added a new "Pull Requests" admin page with search, filtering, and detailed PR viewing
  • Introduced utility classes (Constants and Utils) for better code organization
  • Removed weekly S3 history cleanup cron job and consolidated to hourly cleanup only

Reviewed Changes

Copilot reviewed 34 out of 34 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
webpack.config.js Added webpack entry for pull-requests admin page
inc/classes/rest/class-github-pull-requests.php New REST API endpoint for fetching GitHub pull requests with pagination and search
inc/classes/settings/class-shared-sites.php Added pull requests menu item for governing sites
inc/classes/plugin-configs/class-constants.php New constants class to centralize option names
inc/classes/class-utils.php New utility class with helper methods for site types and S3
assets/src/admin/pull-requests/index.js New React component for pull requests management interface
oneupdate.php Version bump and cron job cleanup changes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Utsav Patel <75293077+up1512001@users.noreply.github.com>
Co-authored-by: Vishal Kakadiya <vishalkakadiya123@gmail.com>
@up1512001 up1512001 force-pushed the feature/pull-requests branch from fed59e3 to a2bffd0 Compare September 19, 2025 08:53
@up1512001 up1512001 requested a review from Copilot September 19, 2025 10:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 34 out of 34 changed files in this pull request and generated 5 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

vishalkakadiya
vishalkakadiya previously approved these changes Sep 19, 2025
Co-authored-by: Utsav Patel <75293077+up1512001@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@up1512001 up1512001 merged commit ee6bd19 into main Sep 19, 2025
5 checks passed
@up1512001 up1512001 deleted the feature/pull-requests branch September 19, 2025 11:24
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.

3 participants