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

using RequestOperationType.PROMOTE for promoteSchema #1569

Merged
merged 8 commits into from
Aug 8, 2023
Merged

using RequestOperationType.PROMOTE for promoteSchema #1569

merged 8 commits into from
Aug 8, 2023

Conversation

Sadaf-A
Copy link
Contributor

@Sadaf-A Sadaf-A commented Aug 4, 2023

About this change - using PROMOTE REQUEST_OPERATION_TYPE for promoteSchemas instead of CREATE

Resolves: #1546

@aindriu-aiven
Copy link
Contributor

aindriu-aiven commented Aug 7, 2023

Hey @Sadaf-A this looks great and works exactly as expected, I noticed one issue which I hadn't included in the original ticket but popped up when I was giving it a quick test.

Under mySchemaRequests and under execSchemas (see links to code below) the PROMOTE enum hadn't been used before so the Request Type is missing from the card.
image

This can be changed by altering the html


<div ng-show="schemaRequest.requestOperationType == 'CREATE'" class="p-2 border-right" style="width:12%">
											<h6 class="text-primary">Request Type</h6><b>
											<span class="badge badge-info">{{ schemaRequest.requestOperationType }} Schema</span></b>
										</div>
										<div ng-show="schemaRequest.requestOperationType == 'UPDATE'" class="p-2 border-right" style="width:12%">
											<h6 class="text-primary">Request Type</h6><b>
											<span class="badge badge-primary">{{ schemaRequest.requestOperationType }} Schema</span></b>
										</div>
										<div ng-show="schemaRequest.requestOperationType == 'DELETE'" class="p-2 border-right" style="width:12%">
											<h6 class="text-primary">Request Type</h6><b>
											<span class="badge badge-danger">{{ schemaRequest.requestOperationType }} Schema</span></b>
										</div>

to


<div ng-show="schemaRequest.requestOperationType !== 'APPROVED'" class="p-2 border-right" style="width:12%">
											<h6 class="text-primary">Request Type</h6><b>
											<span class="badge badge-info">{{ schemaRequest.requestOperationType }} Schema</span></b>
										</div>

<div ng-show="schemaRequest.requestOperationType == 'CREATE'" class="p-2 border-right" style="width:12%">

<div ng-show="schemaRequest.requestOperationType == 'CREATE'" class="p-2 border-right" style="width:12%">

If you aren't comfortable working with the html updates though I am happy to make them.

Change looks great though thanks so much for contributing!
Thanks,
Aindriú

Copy link
Contributor

@aindriu-aiven aindriu-aiven left a comment

Choose a reason for hiding this comment

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

Just added one additional request in another comment

@Sadaf-A
Copy link
Contributor Author

Sadaf-A commented Aug 7, 2023

@aindriu-aiven Thanks I'm glad to know that I was of help. I've also made the changes to the html files as requested by you.

@aindriu-aiven
Copy link
Contributor

@Sadaf-A Great thanks a million I will give this another test and review today!

Copy link
Contributor

@aindriu-aiven aindriu-aiven 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.

Tested schema promotion and the correct RequestOperationType is now being used to describe the operation, the schema is being correctly provisioned to the cluster and all of the ids and versions are correctly appearing in the schema view.

Also worth noting that the request type is displaying correctly on the approval and the and myschemaRequests page.

@aindriu-aiven aindriu-aiven merged commit db1d97e into Aiven-Open:main Aug 8, 2023
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.

Promoted Schema Requests should use the PROMOTE REQUEST_OPERATION_TYPE
2 participants