Skip to content

fix(topic): remove partition update code#526

Merged
rkpattnaik780 merged 2 commits intomainfrom
remove-partition-update-logic
Apr 1, 2021
Merged

fix(topic): remove partition update code#526
rkpattnaik780 merged 2 commits intomainfrom
remove-partition-update-logic

Conversation

@craicoverflow
Copy link
Contributor

Description

Follows up on bf2fc6cc711aee1a0c2a/kafka-admin-api#26 where the OpenAPI schema type was different to the server implementation, causing input to fail.

Verification Steps

  1. Run rhoas kafka topic update on a topic
  2. The update should be successful.
  3. Try to edit this item
  4. Check if in the left menu the feature X is not so long present.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • Documentation added for the feature
  • CI and all relevant tests are passing
  • Code Review completed
  • Verified independently by reviewer

@craicoverflow
Copy link
Contributor Author

@rkpattnaik780 when the backend is back up and running can you verify if this works, if it does, please merge!

@rkpattnaik780 rkpattnaik780 merged commit 044ce5a into main Apr 1, 2021
@rkpattnaik780 rkpattnaik780 deleted the remove-partition-update-logic branch April 1, 2021 11:36
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.

2 participants