Skip to content

Alerting only when we check for repo's#465

Merged
derrickpersson merged 1 commit into
releases/v0.5.5from
hotfix-no-remote-notification
Jul 22, 2022
Merged

Alerting only when we check for repo's#465
derrickpersson merged 1 commit into
releases/v0.5.5from
hotfix-no-remote-notification

Conversation

@derrickpersson
Copy link
Copy Markdown
Contributor

Description

  • We fire the other method too often resulting in a large amount of error messages. Switching so that we only alert them when they focus the codex tab which is when the information is important.

Type of Change

  • Tech Debt (refactoring, unit tests, CI changes, pre-commit hooks, etc)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Test Configuration:

  • CCL version/branch: 1.X
  • API version/branch: 1.X

Checklist:

  • I have performed a self-review of my code
  • I have used codex to provide context, particularly in hard-to-understand areas
  • Any dependent changes have been made available in downstream modules

Screenshots:

@derrickpersson derrickpersson requested a review from saumilptl July 22, 2022 17:29
@derrickpersson derrickpersson merged commit a957c1a into releases/v0.5.5 Jul 22, 2022
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