Skip to content

897: Overrided codebloom.scheduling.enabled and codebloom.notify.enab…#885

Open
Arshadul-Monir wants to merge 1 commit intomainfrom
897
Open

897: Overrided codebloom.scheduling.enabled and codebloom.notify.enab…#885
Arshadul-Monir wants to merge 1 commit intomainfrom
897

Conversation

@Arshadul-Monir
Copy link
Copy Markdown
Collaborator

@Arshadul-Monir Arshadul-Monir commented Apr 2, 2026

897

…led for staging environment

Description of changes

Overridden application.yml values for staging

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

@Arshadul-Monir
Copy link
Copy Markdown
Collaborator Author

/deploy

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

PR Description Required

Please fill out the Description of changes section of the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Configuration Intent

The PR sets codebloom.scheduling.enabled and codebloom.notify.enabled to false for the staging environment. The reviewer should confirm if this is the intended permanent configuration for staging, or if this is a temporary change related to the Notion task "Test setting scheduling and notify to false". Understanding the long-term implications of disabling these features in staging is crucial.

scheduling:
  enabled: false
notify:
  enabled: false

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Ticket Validation Failed

The following issues were found with the attached ticket:

  • Ticket does not have a feature assigned.
  • Ticket does not have a priority defined.

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.

1 participant