Skip to content

Conversation

@Deaponn
Copy link
Collaborator

@Deaponn Deaponn commented Jun 23, 2025

  • branch and/or PR name(s) includes JIRA ID
  • issue "Status" is set to "In review"
  • PR labels are selected

@Deaponn Deaponn requested a review from graduta as a code owner June 23, 2025 11:03
@Deaponn Deaponn changed the base branch from dev to feature/CNF/OGUI-1698/implement-configuration-endpoint June 23, 2025 14:06
@Deaponn Deaponn changed the base branch from feature/CNF/OGUI-1698/implement-configuration-endpoint to dev June 23, 2025 14:08
@Deaponn Deaponn requested review from piechnikk and sKrzysieK June 23, 2025 14:11
@Deaponn Deaponn changed the base branch from dev to feature/CNF/OGUI-1698/implement-configuration-endpoint June 23, 2025 14:12
@Deaponn Deaponn marked this pull request as draft June 30, 2025 13:49
Base automatically changed from feature/CNF/OGUI-1698/implement-configuration-endpoint to dev August 28, 2025 10:28
@Deaponn Deaponn marked this pull request as ready for review October 31, 2025 14:54
Copy link
Collaborator

@sKrzysieK sKrzysieK left a comment

Choose a reason for hiding this comment

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

Generally LGTM but there are a few things I'd like to be clarified

Copy link
Member

@graduta graduta left a comment

Choose a reason for hiding this comment

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

For Control project, we limit ourselves to using ' single quotes for strings, imports, etc. Please update so across your PR

@Deaponn Deaponn requested review from graduta and sKrzysieK November 17, 2025 21:13
sKrzysieK
sKrzysieK previously approved these changes Nov 19, 2025
Copy link
Collaborator

@sKrzysieK sKrzysieK left a comment

Choose a reason for hiding this comment

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

Looks good to me

@graduta graduta merged commit 75b828e into dev Nov 20, 2025
9 checks passed
@graduta graduta deleted the feature/CNF/OGUI-1729/restrictions-endpoint branch November 20, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants