mirror of
https://github.com/koel/koel.git
synced 2026-04-25 16:56:02 +03:00
[PR #1873] [MERGED] feat: use PHP attribute to mark requests disabled in demo #1929
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#1929
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/koel/koel/pull/1873
Author: @phanan
Created: 11/8/2024
Status: ✅ Merged
Merged: 11/8/2024
Merged by: @phanan
Base:
master← Head:attr-for-demo📝 Commits (1)
675b99ffeat: use PHP attribute to mark requests disabled in demo📊 Changes
10 files changed (+104 additions, -14 deletions)
View changed files
➕
app/Attributes/DisabledInDemo.php(+14 -0)📝
app/Http/Controllers/API/ForgotPasswordController.php(+2 -2)📝
app/Http/Controllers/API/Podcast/PodcastController.php(+2 -2)📝
app/Http/Controllers/API/ProfileController.php(+2 -2)📝
app/Http/Controllers/API/ResetPasswordController.php(+2 -2)📝
app/Http/Controllers/Controller.php(+0 -6)📝
app/Http/Kernel.php(+3 -0)➕
app/Http/Middleware/HandleDemoMode.php(+44 -0)📝
tests/Feature/ForgotPasswordTest.php(+21 -0)📝
tests/Feature/ProfileTest.php(+14 -0)📄 Description
Use
#[DisabledInDemo]to mark controller class/action that should be disabled in demo mode.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.