Add Mock Implementation Redirects to Add Dynamic Endpoint UI
Description
Clicking the Add Mock Implementation button redirects to the Add Dynamic Endpoint UI.
Steps to Reproduce
- Create an API without endpoints .
- Go to the Endpoints tab under the API Configurations.
- Click the Add button on the Mock Implementation Card.
Version
4.5.0-beta
Environment Details (with versions)
No response
This is reproducible in the latest built pack as well.
.
PR submitted to fix this: wso2/apim-apps#1153. Please have a look when convenient. Thanks!
@iff-sal I will assign this issue to you as you have already sent a PR. Appreciate if next time you can make sure to get the issue assigned to you before starting to work on it to be fair to others who have already requested to be assigned. Thank you for your contribution.
@tharikaGitHub Thank you! I’ll make sure to get the issue assigned next time 😊. Whenever you get a chance, please review my PR. Thanks again!
Hi @tharikaGitHub, any update regarding this issue and pr(https://github.com/wso2/apim-apps/pull/1153) — it's been a little while and I wanted to check if there's anything else needed from my side or if my pr is ok. Thanks! 😊
Hi @iff-sal we will review your PR soon and let you know if anything need to change. Thank you for your contribution.
Thanks for the update @tharikaGitHub, Happy to rebase or make any changes if needed 😊.
Hi @iff-sal , Thank you for your patience with the delay in reviewing. I’ve added some comments to the PR
@RusJaI Thanks for reviewing my pr, i will fix those changes.
Hi @iff-sal , Appreciate your effort! I’ve requested a few changes. Once those are addressed, we’ll be happy to accept the fix. Thank you!
@RusJaI Thanks for reviewing and approving the changes.
Hi @iff-sal Thank you for your contribution! We are currently in a code freeze period due to an on going product release. Therefore we will get your PR merged soon when possible.