Typos, Bugfixes and explicit response for organization show page

app/Models/Organization.php

@@ -27,8 +27,9 @@ use Laravel\Jetstream\Team as JetstreamTeam;
  * @property User $owner
  * @property Carbon|null $created_at
  * @property Carbon|null $updated_at
- * @property Collection<User> $users
- * @property Collection<string, User> $realUsers
+ * @property Collection<int, User> $users
+ * @property Collection<int, User> $realUsers
+ * @property-read Collection<int, OrganizationInvitation> $teamInvitations
  * @property Membership $membership
  *
  * @method HasMany<OrganizationInvitation> teamInvitations()
@@ -83,7 +84,7 @@ class Organization extends JetstreamTeam
     /**
      * Get all the non-placeholder users of the organization including its owner.
      *
-     * @return Collection<string, User>
+     * @return Collection<int, User>
      */
     public function allRealUsers(): Collection
     {

app/Models/Project.php

@@ -22,7 +22,7 @@ use Illuminate\Database\Eloquent\Relations\HasMany;
  * @property int|null $billable_rate
  * @property-read Organization $organization
  * @property-read Client|null $client
- * @property-read Collection<Task> $tasks
+ * @property-read Collection<int, Task> $tasks
  *
  * @method Builder<Project> visibleByUser(User $user)
  * @method static ProjectFactory factory()

app/Models/Task.php

@@ -22,7 +22,7 @@ use Illuminate\Support\Carbon;
  * @property Carbon|null $updated_at
  * @property-read Project $project
  * @property-read Organization $organization
- * @property-read Collection<TimeEntry> $timeEntries
+ * @property-read Collection<int, TimeEntry> $timeEntries
  *
  * @method static TaskFactory factory()
  */

app/Providers/FortifyServiceProvider.php

@@ -8,8 +8,10 @@ use App\Actions\Fortify\CreateNewUser;
 use App\Actions\Fortify\ResetUserPassword;
 use App\Actions\Fortify\UpdateUserPassword;
 use App\Actions\Fortify\UpdateUserProfileInformation;
+use App\Models\User;
 use Illuminate\Cache\RateLimiting\Limit;
 use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Hash;
 use Illuminate\Support\Facades\RateLimiter;
 use Illuminate\Support\ServiceProvider;
 use Illuminate\Support\Str;
@@ -35,6 +37,20 @@ class FortifyServiceProvider extends ServiceProvider
         Fortify::updateUserPasswordsUsing(UpdateUserPassword::class);
         Fortify::resetUserPasswordsUsing(ResetUserPassword::class);
 
+        Fortify::authenticateUsing(function (Request $request): ?User {
+            /** @var User|null $user */
+            $user = User::query()
+                ->where('email', $request->email)
+                ->where('is_placeholder', '=', false)
+                ->first();
+
+            if ($user !== null && Hash::check($request->password, $user->password)) {
+                return $user;
+            }
+
+            return null;
+        });
+
         RateLimiter::for('login', function (Request $request) {
             $throttleKey = Str::transliterate(Str::lower($request->input(Fortify::username())).'|'.$request->ip());
 

app/Providers/JetstreamServiceProvider.php

@@ -16,6 +16,7 @@ use App\Enums\Role;
 use App\Enums\Weekday;
 use App\Models\Organization;
 use App\Models\OrganizationInvitation;
+use App\Models\User;
 use App\Service\TimezoneService;
 use Brick\Money\Currency;
 use Brick\Money\ISOCurrencyProvider;
@@ -103,7 +104,7 @@ class JetstreamServiceProvider extends ServiceProvider
             'members:change-role',
             'members:update',
             'members:delete',
-        ])->description('Owner users can perform any action.');
+        ])->description('Owner users can perform any action. There is only one owner per organization.');
 
         Jetstream::role(Role::Admin->value, 'Administrator', [
             'projects:view',
@@ -145,7 +146,7 @@ class JetstreamServiceProvider extends ServiceProvider
             'invitations:remove',
             'members:view',
             'members:invite-placeholder',
-        ])->description('Administrator users can perform any action.');
+        ])->description('Administrator users can perform any action, except accessing the billing dashboard.');
 
         Jetstream::role(Role::Manager->value, 'Manager', [
             'projects:view',
@@ -181,7 +182,7 @@ class JetstreamServiceProvider extends ServiceProvider
             'organizations:view',
             'invitations:view',
             'members:view',
-        ])->description('Managers have the ability to read, create, and update their own time entries as well as those of their team.');
+        ])->description('Managers have full access to all projects, time entries, ect. but cannot manage the organization (add/remove member, edit the organization, ect.).');
 
         Jetstream::role(Role::Employee->value, 'Employee', [
             'projects:view',
@@ -192,7 +193,7 @@ class JetstreamServiceProvider extends ServiceProvider
             'time-entries:update:own',
             'time-entries:delete:own',
             'organizations:view',
-        ])->description('Employees have the ability to read, create, and update their own time entries.');
+        ])->description('Employees have the ability to read, create, and update their own time entries and they can see the projects that they are members of.');
 
         Jetstream::role(Role::Placeholder->value, 'Placeholder', [
         ])->description('Placeholders are used for importing data. They cannot log in and have no permissions.');
@@ -210,7 +211,41 @@ class JetstreamServiceProvider extends ServiceProvider
             ->whenRendering(
                 'Teams/Show',
                 function (Request $request, array $data): array {
+                    /** @var Organization $teamModel */
+                    $teamModel = $data['team'];
+                    $owner = $teamModel->owner;
+
                     return array_merge($data, [
+                        'team' => [
+                            'id' => $teamModel->getKey(),
+                            'name' => $teamModel->name,
+                            'currency' => $teamModel->currency,
+                            'owner' => [
+                                'id' => $owner->getKey(),
+                                'name' => $owner->name,
+                                'email' => $owner->email,
+                                'profile_photo_url' => $owner->profile_photo_url,
+                            ],
+                            'users' => $teamModel->users->map(function (User $user): array {
+                                return [
+                                    'id' => $user->getKey(),
+                                    'name' => $user->name,
+                                    'email' => $user->email,
+                                    'profile_photo_url' => $user->profile_photo_url,
+                                    'membership' => [
+                                        'id' => $user->membership->id,
+                                        'role' => $user->membership->role,
+                                    ],
+                                ];
+                            }),
+                            'team_invitations' => $teamModel->teamInvitations->map(function (OrganizationInvitation $invitation): array {
+                                return [
+                                    'id' => $invitation->getKey(),
+                                    'email' => $invitation->email,
+                                    'role' => $invitation->role,
+                                ];
+                            }),
+                        ],
                         'currencies' => array_map(function (Currency $currency): string {
                             return $currency->getName();
                         }, ISOCurrencyProvider::getInstance()->getAvailableCurrencies()),

app/Service/Import/Importers/ClockifyTimeEntriesImporter.php

@@ -162,12 +162,12 @@ class ClockifyTimeEntriesImporter extends DefaultImporter
     #[\Override]
     public function getName(): string
     {
-        return __('importer.toggl_data_importer.name');
+        return __('importer.clockify_time_entries.name');
     }
 
     #[\Override]
     public function getDescription(): string
     {
-        return __('importer.toggl_data_importer.description');
+        return __('importer.clockify_time_entries.description');
     }
 }