Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FOGL-8115: Add Control Entry-points API Flow #332

Merged
merged 9 commits into from
Oct 3, 2023
Merged

Conversation

MonikaSharma06
Copy link
Contributor

@MonikaSharma06 MonikaSharma06 commented Sep 28, 2023

Form validation related changes and other css fixes

@@ -419,7 +416,7 @@
<div class="field">
<div class="columns">
<div class="column is-2">
<div *ngIf="editMode && allowExecute" class="field has-addons is-fullwidth">
<div *ngIf="editMode && af.permitted" class="field has-addons is-fullwidth">
Copy link
Member

Choose a reason for hiding this comment

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

All conditionals stuff with edit mode for permitted needs proper testing; for each role based user; and not auth mode.

@MonikaSharma06 MonikaSharma06 changed the title FOGL-8115: Form validation related changes and other css fixes FOGL-8115: Add Control Entry-points API Flow Sep 28, 2023
Copy link
Member

@praveen-garg praveen-garg left a comment

Choose a reason for hiding this comment

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

Override variables modal has different modal body appearance, from list vs details page; Also contents are not vertically centre.

@MonikaSharma06 MonikaSharma06 merged commit 551713f into develop Oct 3, 2023
@MonikaSharma06 MonikaSharma06 deleted the FOGL-8115 branch October 3, 2023 12:39
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.

3 participants