Skip to content

🛂 feat : added oauth2 implementation#16

Merged
dev-madhurendra merged 3 commits intomainfrom
FE_BE_OAuth2_Integration
Aug 15, 2025
Merged

🛂 feat : added oauth2 implementation#16
dev-madhurendra merged 3 commits intomainfrom
FE_BE_OAuth2_Integration

Conversation

@dev-madhurendra
Copy link
Copy Markdown
Owner

@dev-madhurendra dev-madhurendra commented Aug 11, 2025

📦 Pull Request

📌 Description

Fixes #

🧩 Type of Change

Please delete options that are not relevant.

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Code refactor
  • 🧪 Test addition/changes
  • 📝 Documentation update
  • 🚀 Performance improvement
  • 🔧 Build/deployment changes
  • ⚠️ Breaking change

✅ Checklist

  • I have followed the project's coding standards
  • I have tested the changes locally
  • All dependent packages are updated (if needed)
  • I have added or updated relevant documentation/comments
  • I have added tests that prove my fix is effective or that my feature works
  • All new and existing tests passed
  • I have linked the related issue (if applicable)

🧪 How to Test

  1. Start backend server
  2. Run frontend with npm run dev
  3. Test /api/barber/token endpoint manually or using Postman
  4. [Optional] Check console logs or database

📸 Screenshots / Video

🔍 Review Notes

  • Focus on validation logic in BarberTokenController
  • Ensure role-based access is working correctly

🚨 Breaking Changes?

  • Yes (please describe below)
  • No

If yes, please describe:

📚 Additional Context

@dev-madhurendra dev-madhurendra self-assigned this Aug 11, 2025
@dev-madhurendra dev-madhurendra added the enhancement New feature or request label Aug 11, 2025
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
12.9% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link
Copy Markdown

Comment thread frontend/src/components/molecules/LoginForm/index.tsx
Comment thread backend/src/main/resources/application.yml
@dev-madhurendra dev-madhurendra merged commit fc4f800 into main Aug 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants