Skip to content

Issue 6 individual event pages #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
saltyypringle merged 23 commits into from
Dec 13, 2025
Merged

Issue 6 individual event pages #29

saltyypringle merged 23 commits into from
Dec 13, 2025

Conversation

@Yosuke-95
Copy link
Collaborator

@Yosuke-95 Yosuke-95 commented Dec 10, 2025 •
edited by github-actions bot
Loading

Change Summary

・Implemented Individual Event Pages for Issue #6
・Added the Event model and serializer on the backend
・Created basic tests for the Event model
・Built the frontend event detail page
・Adjusted responsive design

Change Form

  • The pull request title has an issue number
  • The change works by "Smoke testing" or quick testing
  • The change has tests
  • The change has documentation - not sure if anything is needed to document

Other Information

N/A

Related issue

saltyypringle and others added 19 commits November 29, 2025 14:36
@saltyypringle
added the events model with specified fields
8f887c1
@saltyypringle
added some tests for the eventmodel
660835b
@samjjacko
Merge remote-tracking branch 'origin' into issue-6-Individual_event_p…
ec557ee 
…ages
@Yosuke-95
fixed code style error
0b38216
@Yosuke-95
add magic method for event class
8aa58b5
@Yosuke-95
add test case for image file
2a80567
@Yosuke-95
Individual event page template
e22332f
@Yosuke-95
get image from localhost
6e8fbb9
@Yosuke-95
create api to get event information
318159b
@Yosuke-95
get event data from api
5183e14
@pvkudr
Change imports in vies.py and serializerss.py + add some colsole.log …
d5ddd52 
…for testing
@Yosuke-95
add date and startTiime colum
8e4db5b
@pvkudr
Change display of errors
77929f4
@pvkudr
1st hook version
78bbfe0
@pvkudr
Change useEvent hook
940fe9c
@Yosuke-95
changed date into datetime
aaf64cf
@Yosuke-95
Define the API within the game_dev folder
b0f4e34
@Yosuke-95
api/event folder was not necesary
485c1f2
@pvkudr
Design changes - remove some inner divs
3cd2a18
laurenpudz
laurenpudz previously requested changes Dec 11, 2025
View reviewed changes
Copy link
Contributor

@laurenpudz laurenpudz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A really great start, just some minor things to fix up

SafetyInObscurity
SafetyInObscurity requested changes Dec 13, 2025
View reviewed changes
SafetyInObscurity
SafetyInObscurity approved these changes Dec 13, 2025
View reviewed changes
Copy link
Collaborator

@SafetyInObscurity SafetyInObscurity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good guys, great work!

@SafetyInObscurity SafetyInObscurity dismissed laurenpudz’s stale review December 13, 2025 07:30

Issues have been addressed

@saltyypringle saltyypringle merged commit f9a226e into main Dec 13, 2025
5 checks passed
@laurenpudz laurenpudz mentioned this pull request Jan 8, 2026
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SafetyInObscurity SafetyInObscurity SafetyInObscurity approved these changes

@laurenpudz laurenpudz laurenpudz left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Individual event pages

7 participants

@Yosuke-95 @SafetyInObscurity @laurenpudz @saltyypringle @samjjacko @pvkudr