Skip to content

(MODULES-11613) Set permission variable in permission sql EPP#500

Merged
shubhamshinde360 merged 1 commit intomainfrom
MODULES-11613-set-permissions-var
Mar 4, 2026
Merged

(MODULES-11613) Set permission variable in permission sql EPP#500
shubhamshinde360 merged 1 commit intomainfrom
MODULES-11613-set-permissions-var

Conversation

@shubhamshinde360
Copy link
Contributor

@shubhamshinde360 shubhamshinde360 commented Feb 27, 2026

The permission variable is not set in the template file 'templates/create/login/permission.sql.epp'. This creates a faulty tsql query when creating the login resource..
This commits sets the permission variable to fix this issue.

@shubhamshinde360 shubhamshinde360 force-pushed the MODULES-11613-set-permissions-var branch from 971f303 to 957fa54 Compare February 27, 2026 16:34
@shubhamshinde360 shubhamshinde360 marked this pull request as ready for review March 2, 2026 05:50
@shubhamshinde360 shubhamshinde360 requested a review from a team as a code owner March 2, 2026 05:50
@shubhamshinde360 shubhamshinde360 force-pushed the MODULES-11613-set-permissions-var branch from 957fa54 to f71f730 Compare March 2, 2026 12:25
@span786
Copy link
Contributor

span786 commented Mar 2, 2026

@shubhamshinde360 Please add some description.

The permission variable is not set in the template file 'templates/create/login/permission.sql.epp'. This creates a faulty tsql query when creating the login resource..
This commits sets the permission variable to fix this issue.
@shubhamshinde360 shubhamshinde360 force-pushed the MODULES-11613-set-permissions-var branch from f71f730 to f7ca7a6 Compare March 2, 2026 13:57
@shubhamshinde360 shubhamshinde360 merged commit 5e366a5 into main Mar 4, 2026
4 of 7 checks passed
@shubhamshinde360 shubhamshinde360 deleted the MODULES-11613-set-permissions-var branch March 4, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants