Skip to content

Google login and register - #255

Merged
davidperezgar merged 12 commits into
trunkfrom
google-login-register
Sep 3, 2026
Merged

Google login and register#255
davidperezgar merged 12 commits into
trunkfrom
google-login-register

Conversation

@Castellon-ACM

@Castellon-ACM Castellon-ACM commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Fixes #246

Open WordPress Playground Preview

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@davidperezgar
davidperezgar self-requested a review August 21, 2026 11:43
davidperezgar and others added 8 commits August 28, 2026 14:06
This is a pre-release build of google-login-register (Google Sign-In)
merged with the latest trunk, meant for manual install on production
to validate before merging. readme.txt's Stable tag is intentionally
left untouched — this is not a WordPress.org release.
trunk's tabbed settings rebuild replaced the old "render every
registered section automatically" loop with a fixed, hardcoded list of
tab panels. The Google Sign-In section still registers correctly via
the documented frontblocks_register_settings extension hook, but no
tab panel called render_section_if_exists() for it, so it silently
never rendered anywhere after merging trunk in.

Placed outside the main settings <form>, matching the License tab:
Google Sign-In manages its own form/nonce/save handling rather than
going through the shared frontblocks_settings option.
- Add docs/GOOGLE-SIGNIN.md covering settings, the sign-in flow
  (button -> Google Identity Services -> server-side ID token
  verification -> account matching/creation), security notes, and the
  two extension hooks it fires.
- Link it from readme.md and add a Functionalities bullet.
- Add a readme.txt feature description and a 1.6.0 changelog entry.

All following AGENTS.md's "Adding New Features" documentation
checklist, in English per the plugin's language convention.
Keeps it a patch bump on top of the current 1.5.2, matching the
1.5.3-beta.1 version already used for the test build.
@davidperezgar

Copy link
Copy Markdown
Contributor

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-03T11:12:33.987771Z 564ef74 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

# Conflicts:
#	includes/Admin/Settings.php
#	readme.md

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 564ef74580

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread includes/GoogleSignIn/TokenVerifier.php
Comment thread includes/GoogleSignIn/GoogleSignIn.php Outdated
Comment thread includes/Admin/RedundantPlugins.php Outdated
Comment thread includes/GoogleSignIn/GoogleSignIn.php Outdated
Comment thread includes/GoogleSignIn/GoogleSignIn.php
Comment thread includes/GoogleSignIn/block/block.json
Comment thread includes/GoogleSignIn/Settings.php
Comment thread includes/GoogleSignIn/Settings.php
Comment thread includes/Admin/Settings.php
Comment thread includes/GoogleSignIn/GoogleSignIn.php
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@davidperezgar
davidperezgar merged commit 4c64976 into trunk Sep 3, 2026
7 checks passed
@davidperezgar
davidperezgar deleted the google-login-register branch September 3, 2026 11:40
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.

Add "Login with Google" (Google OAuth Sign-In)

2 participants