Merge pull request #53 from thyseus/enabled-function

Make the enabled() function required

Author: bramr94

README.md

@@ -35,7 +35,7 @@ In the `users` method you can define the users (note: the users must exist), the
 // ...
 ->plugins([
     FilamentDeveloperLoginsPlugin::make()
-        ->enabled()
+        ->enabled(app()->environment('local'))
         ->users([
             'Admin' => 'admin@example.com',
             'User' => 'user@example.com',
@@ -65,7 +65,7 @@ Example:
 ```php
 // ...
 FilamentDeveloperLoginsPlugin::make()
-    ->enabled(app()->environment('local'))
+    ->enabled(fn() => app()->environment('local'))
 ```
 
 ### switchable()

src/FilamentDeveloperLoginsPlugin.php

@@ -84,7 +84,7 @@ public static function getById(string $panelId): static
         throw new ImplementationException('No panel found with filament-developer-logins plugin.');
     }
 
-    public function enabled(Closure | bool $value = true): static
+    public function enabled(Closure | bool $value = false): static
     {
         $this->enabled = $value;
 

tests/Feature/DeveloperLoginsControllerTest.php

@@ -52,7 +52,7 @@ public function test_an_exception_is_thrown_when_the_email_is_not_registered():
     public function test_403_is_returned_when_plugin_is_disabled(): void
     {
         FilamentDeveloperLoginsPlugin::current()
-            ->enabled(false);
+            ->enabled();
 
         $user = TestUser::factory()->create([
             'email' => 'developer@dutchcodingcompany.com',

tests/TestCase.php

@@ -55,7 +55,7 @@ protected function registerTestPanels(): void
                 ])
                 ->plugins([
                     FilamentDeveloperLoginsPlugin::make()
-                        ->enabled()
+                        ->enabled(true)
                         ->users([
                             'Administrator' => 'developer@dutchcodingcompany.com',
                         ])