✨ move trip creation from closed to open beta

see #2411
Traewelling · Mar 17, 2024 · 3ccb58c · 3ccb58c
1 parent 914f8a3
commit 3ccb58c
Show file tree

Hide file tree

Showing 10 changed files with 72 additions and 58 deletions.
diff --git a/lang/de.json b/lang/de.json
@@ -758,5 +758,6 @@
     "error.419": "Zeitüberschreitung",
     "error.429": "Zu viele Anfragen",
     "error.500": "Interner Serverfehler",
-    "support.rate_limit_exceeded": "Du hast vor kurzem bereits eine Support-Anfrage erstellt. Bitte warte noch etwas, bevor du eine weitere Anfrage erstellst."
+    "support.rate_limit_exceeded": "Du hast vor kurzem bereits eine Support-Anfrage erstellt. Bitte warte noch etwas, bevor du eine weitere Anfrage erstellst.",
+    "create-trip-manually": "Fahrt manuell erstellen"
 }
diff --git a/lang/en.json b/lang/en.json
@@ -758,5 +758,6 @@
     "error.419": "Page Expired",
     "error.429": "Too Many Requests",
     "error.500": "Server Error",
-    "support.rate_limit_exceeded": "You have recently created a support request. Please wait a bit before creating another request."
+    "support.rate_limit_exceeded": "You have recently created a support request. Please wait a bit before creating another request.",
+    "create-trip-manually": "Create trip manually"
 }
diff --git a/resources/views/admin/layout.blade.php b/resources/views/admin/layout.blade.php
@@ -58,10 +58,6 @@ class="brand-image me-3" style="width: 30px; opacity: 0.8">
                                 <i class="fa-solid fa-train"></i>
                                 Checkin
                             </a>
-                            <a class="nav-link" href="{{ route('admin.trip.create') }}">
-                                <i class="fa-solid fa-plus"></i>
-                                Trips
-                            </a>
                             <a class="nav-link" href="{{ route('admin.stations') }}">
                                 <i class="fa-solid fa-map-marker"></i>
                                 Stations

diff --git a/resources/views/admin/trip/create.blade.php b/resources/views/admin/trip/create.blade.php
diff --git a/resources/views/beta/trip-creation.blade.php b/resources/views/beta/trip-creation.blade.php
@@ -0,0 +1,18 @@
+@extends('layouts.app')
+
+@section('title', 'Create trip manually')
+
+@section('content')
+    <div class="container">
+        <div id="trip-creation-form">
+            @if(app()->getLocale() !== 'en')
+                <div class="alert alert-info" role="alert">
+                    <i class="fas fa-info-circle"></i>
+                    {{__('page-only-available-in-language', ['language' => __('language.en')])}}
+                </div>
+            @endif
+
+            <trip-creation-form></trip-creation-form>
+        </div>
+    </div>
+@endsection
diff --git a/resources/views/closed-beta/trip-creation.blade.php b/resources/views/closed-beta/trip-creation.blade.php
diff --git a/resources/views/stats/stats.blade.php b/resources/views/stats/stats.blade.php
@@ -35,23 +35,32 @@
             <div class="col-lg-4">
                 @include('stats.includes.global_cards')
 
-                <h4>
-                    {{__('experimental-features')}}
-                </h4>
-                <ul>
-                    @if(auth()->check() && auth()->user()->hasRole('closed-beta'))
+                @if(auth()->check() && auth()->user()->hasRole('open-beta'))
+                    <h4>
+                        {{__('experimental-features')}}
+                    </h4>
+                    <ul>
+                        @if(auth()->check() && auth()->user()->hasRole('closed-beta'))
+                            <li>
+                                <a href="{{route('stats.stations')}}">
+                                    {{__('stats.stations.description')}}
+                                </a>
+                            </li>
+                        @endif
                         <li>
-                            <a href="{{route('stats.stations')}}">
-                                {{__('stats.stations.description')}}
+                            <a href="{{route('stats.daily', ['dateString' => today()->toDateString()])}}">
+                                {{__('stats.daily.description')}}
                             </a>
                         </li>
-                    @endif
-                    <li>
-                        <a href="{{route('stats.daily', ['dateString' => \Illuminate\Support\Facades\Date::today()->toDateString()])}}">
-                            {{__('stats.daily.description')}}
-                        </a>
-                    </li>
-                </ul>
+                        @can('create-manual-trip')
+                            <li>
+                                <a href="{{route('trip.create')}}">
+                                    {{__('create-trip-manually')}}
+                                </a>
+                            </li>
+                        @endcan
+                    </ul>
+                @endif
             </div>
         </div>
     </div>

diff --git a/resources/vue/components/TripCreation/TripCreationForm.vue b/resources/vue/components/TripCreation/TripCreationForm.vue
@@ -97,7 +97,7 @@ export default {
                         window.location.href = `/trains/trip/?${new URLSearchParams(query).toString()}`;
                     });
                 }
-                if(data.status === 422) {
+                if (data.status === 422) {
                     data.json().then((result) => {
                         alert(result.message);
                     });
@@ -121,20 +121,24 @@ export default {
     <div>
         <h1 class="fs-4">
             <i class="fa fa-plus" aria-hidden="true"></i>
-            Create trip manually (closed-beta)
+            Create trip manually
         </h1>
 
         <div class="alert alert-info">
             <h2 class="fs-5">
                 <i class="fa fa-info-circle" aria-hidden="true"></i>
-                Beta users only
+                Beta
             </h2>
 
-            This form is currently for testing purposes only.
-            Beta users can create a trip with manually entered data.
-            All Users can check in to this trip.
-            It should be tested if the trip is created correctly and all data required for the trip is present, so no
-            (500) errors occur or if features are missing which are not mentioned in the limitations section.
+            Please note, that the creation of trips is currently in beta.
+            You can create trips manually by entering the stations and times.
+            There are some limitations, which are listed below.
+            <br/>
+            If you encounter any issues, please report them to us.
+            <a href="https://github.com/Traewelling/traewelling/issues/new/choose" target="_blank" class="float-end btn btn-sm btn-outline-danger">
+                Report issue
+            </a>
+
         </div>
 
         <form @submit.prevent="sendForm" class="mb-3">
@@ -166,7 +170,8 @@ export default {
             </div>
             <div class="row g-3 mt-1">
                 <div class="col-4">
-                    <input type="text" class="form-control" placeholder="Linie (S1, ICE 13, ...)" v-model="trainTypeInput">
+                    <input type="text" class="form-control" placeholder="Linie (S1, ICE 13, ...)"
+                           v-model="trainTypeInput">
                 </div>
                 <div class="col-4">
                     <input type="text" class="form-control" placeholder="Nummer (optional)" aria-label="Fahrtnummer"
@@ -193,9 +198,14 @@ export default {
             </h2>
 
             <ul>
-                <li>Only stations available in DB-HAFAS are supported</li>
-                <li>Stopovers can't be created yet</li>
-                <li>Polyline is generated straight from origin to destination (Brouter generation will apply if the difference between air distance and distance by train isn't too big)</li>
+                <li>Only stations available in the time table of DB Navigator are supported</li>
+                <li>
+                    The map lines are generated straight from the given stations
+                    <small>(we try to find a route via Brouter, but this can be result in wrong data)</small>
+                </li>
+                <li>The trip is generally created public - so if you checkin to a trip, everyone who can see your status can ride with you.</li>
+                <li>The operator can't be set in this form (possible via API)</li>
+                <li>There are no visible error messages for this form. So, if nothing happens on submit... sorry. There is an error.</li>
             </ul>
         </div>
     </div>

diff --git a/routes/web.php b/routes/web.php
@@ -123,9 +123,11 @@
  * All of these routes can only be used by fully registered users.
  */
 Route::middleware(['auth', 'privacy'])->group(function() {
-    Route::view('/beta/trip-creation', 'closed-beta.trip-creation')
-         ->can('create-manual-trip')
-         ->name('beta.trip-creation');
+
+    Route::redirect('/beta/trip-creation', '/trip/create'); //TODO: remove after 2024-06
+    Route::view('/trip/create', 'beta.trip-creation')
+         ->middleware(['can:create-manual-trip'])
+         ->name('trip.create');
 
     Route::post('/ics/createToken', [IcsController::class, 'createIcsToken'])
          ->name('ics.createToken'); //TODO: Replace with API Endpoint

diff --git a/routes/web/admin.php b/routes/web/admin.php
@@ -43,9 +43,6 @@
         });
 
         Route::prefix('trip')->group(function() {
-            Route::view('/create', 'admin.trip.create')
-                 ->name('admin.trip.create');
-
             Route::get('/{id}', [TripController::class, 'renderTrip'])
                  ->name('admin.trip.show');
         });