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.

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

Adding Early-Bird Role to the Early-Bird user #29

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Sam2004-2
Copy link
Contributor

@Sam2004-2 Sam2004-2 commented Oct 4, 2023

Morning bot PR

What does it do?
Created the Early-Bird role and added implementation for giving said role to the Early-Bird user each day.

Why is it needed?
It is an enhancement listed at #25

Is this change tracked by an issue?
This change is tracked by issue #25

Acceptance Criteria

  • Implementation
  • Test Coverage
  • Change Reflected in Documentation?
  • CI/CD pipelines configured to accomodate change

@coillteoir
Copy link
Owner

Nice work so far. Do these changes check if the role exists? And if it doesn't is there a way to create the role?

@Sam2004-2
Copy link
Contributor Author

They do check if the role exists, I've added the functionality for creating the Early-Bird role

@Sam2004-2
Copy link
Contributor Author

Added the SERVER_ID attribute to deal with creation of the role.

@coillteoir
Copy link
Owner

Nice work, image

Bot is working in the audit log above

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.

Adding an early bird role to the person who was first to say good morning
2 participants