Skip to content

Conversation

@spacecostsmoney
Copy link
Contributor

@spacecostsmoney spacecostsmoney commented Oct 29, 2025

Reverts #45

@spacecostsmoney spacecostsmoney self-assigned this Oct 29, 2025
@jsmilliken jsmilliken requested a review from Copilot October 29, 2025 21:35
@spacecostsmoney spacecostsmoney merged commit 70e4e0b into main Oct 29, 2025
1 of 2 checks passed
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 reverts changes from PR #45, switching back from using Slack channel IDs to using channel names throughout the application.

  • Reverts parameter and variable names from SlackChannelId to SlackChannel
  • Updates default value from channel ID C03HVBJ7WKC to channel name si-sandbox
  • Changes deployment configuration to use goals channel name instead of CF4U95FN0 channel ID

Reviewed Changes

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

Show a summary per file
File Description
test/index.test.js Updates test variables and environment setup to use slackChannel instead of slackChannelId
template.yaml Reverts SAM template parameter from SlackChannelId to SlackChannel with name-based default
src/slack.js Changes function parameter from channelId to channelName
src/slack-service.js Updates function parameters to use channelName instead of channelId
src/index.js Changes environment variable reference from SlackChannelId to SlackChannel
README.md Updates documentation to reflect channel name usage and fixes capitalization
.github/workflows/deploy.yml Updates deployment command to use channel name goals instead of ID CF4U95FN0

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

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.

2 participants