Sicherhaven/Eventify-frontend
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: allow guests to view event details by passing pre-loaded data

Browse Source 
LearnMoreScreen now accepts an optional initialEvent parameter so it
can render immediately from already-loaded data instead of re-fetching
from the event-details API. This fixes the guest-mode flow where the
unauthenticated API call was failing. Also changed getEventDetails to
requiresAuth: true so logged-in users send their token when the API
path is used.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Sicherhaven
2026-03-29 19:25:40 +05:30
parent 1e90f5fc4b
commit c32f343558
6 changed files with 26 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/features/events/services/events_service.dart
View File

@@ -40,7 +40,7 @@ class EventsService {
/// Event details
Future<EventModel> getEventDetails(int eventId) async {
final res = await _api.post(ApiEndpoints.eventDetails, body: {'event_id': eventId}, requiresAuth: false);
final res = await _api.post(ApiEndpoints.eventDetails, body: {'event_id': eventId}, requiresAuth: true);
return EventModel.fromJson(Map<String, dynamic>.from(res));
}

4
lib/screens/calendar_screen.dart
View File

@@ -503,7 +503,7 @@ class _CalendarScreenState extends State<CalendarScreen> {
: (e.startDate != null && e.endDate != null ? '${e.startDate} - ${e.endDate}' : (e.startDate ?? ''));
return GestureDetector(
onTap: () => Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id))),
onTap: () => Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id, initialEvent: e))),
child: Card(
elevation: 6,
margin: const EdgeInsets.fromLTRB(20, 10, 20, 10),
@@ -563,7 +563,7 @@ class _CalendarScreenState extends State<CalendarScreen> {
: (e.startDate ?? ''));
return GestureDetector(
onTap: () => Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id))),
onTap: () => Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id, initialEvent: e))),
child: Container(
margin: const EdgeInsets.fromLTRB(16, 0, 16, 14),
decoration: BoxDecoration(

10
lib/screens/home_desktop_screen.dart
View File

@@ -55,10 +55,10 @@ class _HomeDesktopScreenState extends State<HomeDesktopScreen> {
return const SettingsScreen();
default:
return _HomeContent(
onEventTap: (eventId) {
onEventTap: (eventId, event) {
Navigator.push(
context,
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: eventId)),
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: eventId, initialEvent: event)),
);
},
);
@@ -70,7 +70,7 @@ class _HomeDesktopScreenState extends State<HomeDesktopScreen> {
// Home content — hero, categories, event grid
// ---------------------------------------------------------------------------
class _HomeContent extends StatefulWidget {
final void Function(int eventId) onEventTap;
final void Function(int eventId, EventModel event) onEventTap;
const _HomeContent({required this.onEventTap});
@override
@@ -573,7 +573,7 @@ class _HomeContentState extends State<_HomeContent>
child: GestureDetector(
onTap: () {
Navigator.of(ctx).pop();
widget.onEventTap(event.id);
widget.onEventTap(event.id, event);
},
child: Container(
padding: const EdgeInsets.symmetric(
@@ -753,7 +753,7 @@ class _HomeContentState extends State<_HomeContent>
return MouseRegion(
cursor: SystemMouseCursors.click,
child: GestureDetector(
onTap: () => widget.onEventTap(e.id),
onTap: () => widget.onEventTap(e.id, e),
child: Container(
decoration: BoxDecoration(
color: Colors.white,

14
lib/screens/home_screen.dart
View File

@@ -395,7 +395,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
onTap: () {
Navigator.of(context).pop();
if (ev.id != null) {
Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id)));
Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id, initialEvent: ev)));
}
},
);
@@ -805,7 +805,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
onTap: () {
Navigator.of(context).pop();
if (ev.id != null) {
Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id)));
Navigator.of(context).push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id, initialEvent: ev)));
}
},
child: Container(
@@ -1303,7 +1303,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
onTap: () {
if (event.id != null) {
Navigator.push(context,
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: event.id)));
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: event.id, initialEvent: event)));
}
},
child: Padding(
@@ -1663,7 +1663,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
return GestureDetector(
onTap: () {
if (event.id != null) {
Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: event.id)));
Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: event.id, initialEvent: event)));
}
},
child: Container(
@@ -1855,7 +1855,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
return GestureDetector(
onTap: () {
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id)));
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id, initialEvent: e)));
},
child: Container(
margin: const EdgeInsets.symmetric(vertical: 0),
@@ -1924,7 +1924,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
return GestureDetector(
onTap: () {
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id)));
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id, initialEvent: e)));
},
child: SizedBox(
width: 220,
@@ -2090,7 +2090,7 @@ class _HomeScreenState extends State<HomeScreen> with SingleTickerProviderStateM
return GestureDetector(
onTap: () {
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id)));
if (e.id != null) Navigator.push(context, MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: e.id, initialEvent: e)));
},
child: Container(
margin: const EdgeInsets.only(bottom: 18),

11
lib/screens/learn_more_screen.dart
View File

@@ -15,7 +15,8 @@ import '../core/constants.dart';
class LearnMoreScreen extends StatefulWidget {
final int eventId;
const LearnMoreScreen({Key? key, required this.eventId}) : super(key: key);
final EventModel? initialEvent;
const LearnMoreScreen({Key? key, required this.eventId, this.initialEvent}) : super(key: key);
@override
State<LearnMoreScreen> createState() => _LearnMoreScreenState();
@@ -48,7 +49,13 @@ class _LearnMoreScreenState extends State<LearnMoreScreen> {
void initState() {
super.initState();
_pageNotifier = ValueNotifier(0);
_loadEvent();
if (widget.initialEvent != null) {
_event = widget.initialEvent;
_loading = false;
WidgetsBinding.instance.addPostFrameCallback((_) => _startAutoScroll());
} else {
_loadEvent();
}
}
@override

4
lib/screens/profile_screen.dart
View File

@@ -576,7 +576,7 @@ class _ProfileScreenState extends State<ProfileScreen>
onTap: () {
if (ev.id != null) {
Navigator.of(context)
.push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id)));
.push(MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id, initialEvent: ev)));
}
},
contentPadding: const EdgeInsets.symmetric(horizontal: 12, vertical: 8),
@@ -1388,7 +1388,7 @@ class _ProfileScreenState extends State<ProfileScreen>
onTap: () {
if (ev.id != null) {
Navigator.of(context).push(
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id)),
MaterialPageRoute(builder: (_) => LearnMoreScreen(eventId: ev.id, initialEvent: ev)),
);
}
},