Skip to content

feat: add api remove event#633

Open
quangtuanitmo18 wants to merge 6 commits intomasterfrom
feat-remove-event
Open

feat: add api remove event#633
quangtuanitmo18 wants to merge 6 commits intomasterfrom
feat-remove-event

Conversation

@quangtuanitmo18
Copy link
Copy Markdown
Contributor

Add api to remove event from project

Comment thread src/typeDefs/event.ts Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new GraphQL mutation to remove an event from a project, along with related stored event data in MongoDB.

Changes:

  • Extend the GraphQL schema with a removeEvent(projectId, eventId): Boolean! mutation guarded by @requireUserInWorkspace.
  • Add the removeEvent resolver wiring to call into the events model layer.
  • Implement EventsFactory.removeEvent() to delete the original event plus related repetitions and daily event records.
  • Bump package version to 1.4.8.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
src/typeDefs/event.ts Adds the removeEvent mutation to the GraphQL schema.
src/resolvers/event.js Implements the GraphQL resolver for removeEvent.
src/models/eventsFactory.js Implements MongoDB deletions for event + related collections.
package.json Version bump to reflect the new API feature.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/models/eventsFactory.js
Comment thread src/models/eventsFactory.js
Comment thread src/resolvers/event.js Outdated
Comment thread src/models/eventsFactory.js Outdated
@quangtuanitmo18 quangtuanitmo18 requested a review from neSpecc April 25, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants