Skip to content

feat: Calvin - created EventGuest Repository Class #56

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
Closed
wants to merge 29 commits into from
Closed

feat: Calvin - created EventGuest Repository Class #56

wants to merge 29 commits into from

Conversation

Kaldigi252-ops
Copy link
Contributor

@Kaldigi252-ops Kaldigi252-ops commented Oct 20, 2025 •
edited
Loading

Capstone Team5 - PR - EventGuest Repo Class

Implementation

  • created Event Guest Repo class
  • configured Dev Container to Java version 17

Boyce007
Boyce007 requested changes Oct 21, 2025
View reviewed changes
Copy link
Contributor

@Boyce007 Boyce007 left a comment

Choose a reason for hiding this comment

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

Remove those .DS_Store files, Also we don't test repos directly we create a service class and use the methods from the repo to create our business logic look at the examples in the user repo and service that I created

@vercel
Copy link

vercel bot commented Oct 21, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
cs-25-2-team5 Ready Ready Preview Comment Oct 22, 2025 8:10am

@Kaldigi252-ops Kaldigi252-ops changed the title feat: Calvin - created EventGuestRepo & EventGuestRepoTest Repositories feat: Calvin - created EventGuestRepo Repository Oct 21, 2025
@Kaldigi252-ops Kaldigi252-ops changed the title feat: Calvin - created EventGuestRepo Repository feat: Calvin - created EventGuest Repository Class Oct 21, 2025
@Kaldigi252-ops
Copy link
Contributor Author

Removed .DS_Store files from the repository and added it to '.gitignore' to prevent future tracking.
Also removed the Unit test for Event Guest Repo since it is not needed.

Boyce007
Boyce007 requested changes Oct 21, 2025
View reviewed changes
event-planner/api/demo/src/main/resources/application.properties Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you add this file? We already have a YAML file for configuration

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I searched for any missing configurations and the yaml was suggested.
Perhaps it didn't appear in my pull from main. I'll remove it.

Copy link
Contributor

Choose a reason for hiding this comment

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

You did a pull from main?

Copy link
Contributor

Choose a reason for hiding this comment

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

I changed the application file in this commit 6b37111 it was added yesterday maybe you didn't pull from then

@Boyce007 Boyce007 closed this Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Boyce007 Boyce007 Boyce007 requested changes

@ethanhillman ethanhillman Awaiting requested review from ethanhillman

@tmcrockett252 tmcrockett252 Awaiting requested review from tmcrockett252

@JoyBrown252 JoyBrown252 Awaiting requested review from JoyBrown252

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kaldigi252-ops @Boyce007