Skip to content

Introduce extend command to allow sandbox reservation extension#51

Open
andrzejkupczyk wants to merge 1 commit intomasterfrom
add-extend-command
Open

Introduce extend command to allow sandbox reservation extension#51
andrzejkupczyk wants to merge 1 commit intomasterfrom
add-extend-command

Conversation

@andrzejkupczyk
Copy link
Collaborator

This pull request adds support for extending sandbox bookings in the messaging bot. The main change is the introduction of a new command handler that allows users to extend their sandbox reservation by resetting the due date to five additional working days if they are the current owner.

New sandbox extension feature:

  • Added messages/extend.js handler to process extension requests for sandboxes, including logic to verify ownership, update the booking date, and calculate the new due date based on working days.
  • Registered the new extend handler in messages/index.js so the bot recognizes and responds to extension commands.

Copy link

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

This PR introduces a new "extend" command functionality to the messaging bot that allows users to extend their sandbox reservations for an additional 5 working days.

  • Added a new command handler for sandbox extension requests
  • Updated the bot to recognize and process extension commands
  • Incremented the package version to reflect the new feature

Reviewed Changes

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

File Description
package.json Version bump from 2.1.1 to 2.2.0 for new feature release
messages/index.js Registered the new extend handler to enable command recognition
messages/extend.js New command handler implementing sandbox extension logic with ownership verification

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants