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

VIB-112: Modify auto_remind_checkin mailer for users who need to enter timesheets #299

Merged
merged 3 commits into from
Mar 25, 2025

Conversation

IvanRuskevych
Copy link
Collaborator

@IvanRuskevych IvanRuskevych commented Mar 24, 2025

VIB-112 Powered by Pull Request Badge

Hello Team, please review this PR.

  1. Modified @who_is_waiting message to remind users to fill out their timesheets.
  2. Added user_belongs_to_timesheet_team? method to check team settings.

image
image
image

…r timesheets
@IvanRuskevych IvanRuskevych added the review needed Code reviews needed by the minimum number of developers required label Mar 24, 2025
@IvanRuskevych IvanRuskevych self-assigned this Mar 24, 2025
@rogergraves rogergraves added review done Code reviews are completed by the minimum number of developers required ready for testing Ready for QA to test locally or on a test server and removed review needed Code reviews needed by the minimum number of developers required labels Mar 24, 2025
@mariiapopova22 mariiapopova22 added in testing The QA team is testing this branch and removed ready for testing Ready for QA to test locally or on a test server labels Mar 24, 2025
@rezvaya13 rezvaya13 temporarily deployed to vibereport-pr-299 March 24, 2025 15:39 Inactive
@rogergraves rogergraves temporarily deployed to vibereport-pr-299 March 24, 2025 15:56 Inactive
@rezvaya13 rezvaya13 added qa done QA completed testing successfully and removed in testing The QA team is testing this branch labels Mar 25, 2025
@Maryna-Harasko Maryna-Harasko merged commit de61bbd into master Mar 25, 2025
1 check passed
@Maryna-Harasko Maryna-Harasko deleted the VIB-112 branch March 25, 2025 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qa done QA completed testing successfully review done Code reviews are completed by the minimum number of developers required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants