api-manager icon indicating copy to clipboard operation
api-manager copied to clipboard

Add Mock Implementation Redirects to Add Dynamic Endpoint UI

Open ashanhr opened this issue 10 months ago • 13 comments

Description

Clicking the Add Mock Implementation button redirects to the Add Dynamic Endpoint UI.

Steps to Reproduce

  1. Create an API without endpoints .
  2. Go to the Endpoints tab under the API Configurations.
  3. Click the Add button on the Mock Implementation Card.

Version

4.5.0-beta

Environment Details (with versions)

No response

ashanhr avatar Feb 24 '25 08:02 ashanhr

This is reproducible in the latest built pack as well.

tharikaGitHub avatar Oct 03 '25 08:10 tharikaGitHub

.

JayanaGunaweera01 avatar Oct 04 '25 03:10 JayanaGunaweera01

PR submitted to fix this: wso2/apim-apps#1153. Please have a look when convenient. Thanks!

iff-sal avatar Oct 04 '25 21:10 iff-sal

@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 avatar Oct 06 '25 03:10 tharikaGitHub

@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!

iff-sal avatar Oct 07 '25 13:10 iff-sal

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! 😊

iff-sal avatar Oct 17 '25 13:10 iff-sal

Hi @iff-sal we will review your PR soon and let you know if anything need to change. Thank you for your contribution.

tharikaGitHub avatar Oct 20 '25 05:10 tharikaGitHub

Thanks for the update @tharikaGitHub, Happy to rebase or make any changes if needed 😊.

iff-sal avatar Oct 20 '25 06:10 iff-sal

Hi @iff-sal , Thank you for your patience with the delay in reviewing. I’ve added some comments to the PR

RusJaI avatar Oct 22 '25 06:10 RusJaI

@RusJaI Thanks for reviewing my pr, i will fix those changes.

iff-sal avatar Oct 22 '25 08:10 iff-sal

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 avatar Oct 29 '25 19:10 RusJaI

@RusJaI Thanks for reviewing and approving the changes.

iff-sal avatar Oct 30 '25 15:10 iff-sal

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.

tharikaGitHub avatar Oct 31 '25 10:10 tharikaGitHub