Skip to content

Conversation

@anupama-vm
Copy link
Contributor

PR checklist

TaskId: (https://github.com/SkappHQ/skapp/issues/[id])

Summary

How to test

Project Checklist

  • Changes build without any errors
  • Have written adequate test cases
  • Done developer testing in
    • Chrome
    • Firefox
    • Safari
  • Code is formatted with npm run format
  • Code is linted with npm run check-lint
  • No unnecessary comments left in code
  • Made corresponding changes to the documentation

Other

  • New atomic components added
  • New molecules added
  • New pages(routes) added
  • New dependencies installed

PR Checklist

  • Pull request is raised from the correct source branch
  • Pull request is raised to the correct destination branch
  • Pull request is raised with correct title
  • Pull request is self reviewed
  • Pull request is self assigned
  • Suitable pull request status labels are added (ready-for-code-review)

Additional Information

Advanced the frontend/src/enterprise submodule to commit 2790d046 to incorporate recent changes from the enterprise repository.
Advanced the enterprise submodule pointer to commit 097123ee1314e242462a27fdf439a98e384bc8cf to incorporate recent changes from the submodule repository.
Copilot AI review requested due to automatic review settings July 14, 2025 19:25
@anupama-vm anupama-vm requested a review from ThinuwanW as a code owner July 14, 2025 19:25
@CLAassistant
Copy link

CLAassistant commented Jul 14, 2025

CLA assistant check
All committers have signed the CLA.

This comment was marked as outdated.

@anupama-vm anupama-vm self-assigned this Jul 15, 2025
@sonarqubecloud
Copy link

@anupama-vm anupama-vm changed the base branch from main to develop-improvements July 28, 2025 04:04
…e2e-recaptcha-bypass

# Conflicts:
#	backend/src/main/java/com/skapp/enterprise
#	frontend/src/enterprise
@sonarqubecloud
Copy link

@anupama-vm anupama-vm requested a review from Copilot August 22, 2025 09:52
Copy link
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 implements an e2e reCAPTCHA bypass feature by integrating enterprise-level bypass token validation into the authentication service. The changes update submodule references and add bypass token validation capabilities to the community authentication implementation.

  • Updates enterprise submodule commits for both frontend and backend
  • Adds bypass token validation dependency to the community authentication service

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
frontend/src/enterprise Updates enterprise submodule commit reference
backend/src/main/java/com/skapp/enterprise Updates enterprise submodule commit reference
backend/src/main/java/com/skapp/community/common/service/impl/AuthServiceImpl.java Adds import and dependency injection for BypassTokenValidator

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

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.

2 participants