Skip to content
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

Manager Stuff #26

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Manager Stuff #26

merged 4 commits into from
Sep 10, 2024

Conversation

rutmanz
Copy link
Member

@rutmanz rutmanz commented Sep 10, 2024

  • Show managers in a different color on member grid to help members know who to talk to for guidance
  • Add manager checkin feature for automatic messaging

@rutmanz rutmanz requested a review from sturmanj September 10, 2024 06:15
@rutmanz rutmanz enabled auto-merge September 10, 2024 06:15
@rutmanz rutmanz requested a review from mimizh2418 September 10, 2024 17:35
mimizh2418
mimizh2418 approved these changes Sep 10, 2024
View reviewed changes
Copy link
Member

@mimizh2418 mimizh2418 left a comment

Choose a reason for hiding this comment

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

LGTM (very descriptive branch name thank you zach)

src/slack/blocks/responses.ts Show resolved Hide resolved
@rutmanz rutmanz merged commit a792d11 into main Sep 10, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mimizh2418 mimizh2418 mimizh2418 approved these changes

@sturmanj sturmanj Awaiting requested review from sturmanj

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
@rutmanz @mimizh2418