Skip to content

Fixes final bugs (admin routes only accessible for admin users now)#75

Merged
Kalu32k merged 2 commits intomainfrom
fix-admin-route-protection
Jun 6, 2025
Merged

Fixes final bugs (admin routes only accessible for admin users now)#75
Kalu32k merged 2 commits intomainfrom
fix-admin-route-protection

Conversation

@AlexCode-dot
Copy link
Owner

  • Secured all sensetive api routes (now only admin access)

Pull Request Type

Please check the type of change your PR introduces:

  • Feature (a new feature for the project)
  • Fix (a bug fix)
  • Chore (updating grunt tasks etc; no production code change)
  • Refactor (refactoring production code)
  • Docs (changes to the documentation)
  • Style (formatting, missing semi colons, etc; no production code change)
  • Test (adding missing tests, refactoring tests; no production code change)

Description

[Provide a detailed description of the changes in this pull request]

Motivation and Context

[Why is this change required? What problem does it solve?]

How Has This Been Tested?

[Describe the tests that you ran to verify your changes]

Screenshots (if appropriate):

[Add screenshots here]

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have named this PR according to following naming convention: #[issue-number] PR description

Additional Information:

[Add any other information about the PR here]

Related Issues:

[Link any related issues here]

- Secured all sensetive api routes (now only admin access)
@vercel
Copy link

vercel bot commented Jun 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kino-next-js ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2025 4:50pm

- Updated tests to work with new admin/user logic
@Kalu32k Kalu32k merged commit efcb3fe into main Jun 6, 2025
3 checks passed
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