Skip to content

Conversation

@loadez
Copy link
Contributor

@loadez loadez commented Nov 3, 2025

πŸ“ Description

βœ… Checklist

  • I have tested this change
  • This change requires documentation update

@loadez loadez self-assigned this Nov 3, 2025
@github-project-automation github-project-automation bot moved this to Backlog in Roadmap Nov 3, 2025
@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 66b8e08 to 1dc5968 Compare November 10, 2025 16:00
@loadez loadez marked this pull request as ready for review November 10, 2025 16:00
@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 1dc5968 to 1dd600e Compare November 10, 2025 16:01
@loadez loadez marked this pull request as draft November 10, 2025 16:35
@loadez loadez marked this pull request as ready for review November 11, 2025 14:20
@loadez
Copy link
Contributor Author

loadez commented Nov 19, 2025

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

πŸ’‘ Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with πŸ‘.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@loadez loadez marked this pull request as draft November 19, 2025 12:30
@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 1dd600e to 4abccf9 Compare November 19, 2025 14:51
@loadez
Copy link
Contributor Author

loadez commented Nov 19, 2025

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

πŸ’‘ Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with πŸ‘.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 4abccf9 to 8aaf5c4 Compare November 19, 2025 15:30
@loadez loadez marked this pull request as ready for review November 19, 2025 15:30
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

πŸ’‘ Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with πŸ‘.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 8aaf5c4 to 3869484 Compare November 19, 2025 15:43
@loadez
Copy link
Contributor Author

loadez commented Nov 21, 2025

Hey @DamjanBecirovic @hamir-suspect can you guys take a look at this issue?

The test that failed seems flaky so don't worry about it, I rerun the pipeline and it passed. After I rebase this branch with main the pipeline should run again and we will probably get all greens.

@loadez loadez force-pushed the feature/global-enfoce-whitelist-button branch from 3869484 to faab9e3 Compare November 21, 2025 10:44
@DamjanBecirovic
Copy link
Collaborator

This only saves the whitelist config in the organization settings from the front.
Is the part where we use that config to control whitelis behavior already implemented, or will it come in the next PR?
Either way, please update the PR description with that and any other relevant details and links.

@loadez
Copy link
Contributor Author

loadez commented Nov 21, 2025

It's implemented in another service, this is modification is to enable the user to set this flag to apply the whitelist to branches created before updating the branch/tag whitelist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants