Skip to content

Completing unfinished features #18

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

antoni-g
Copy link

@antoni-g antoni-g commented Jul 27, 2021

Completing development for basic functionality, extending with features that were not available in the base repo

Goal was to allow the use of the UI to add/save/edit mappings.
Still room for improvement, but as is this fork is currently being used in my org for the intended purpose deployed to a cloud instance

  • Enabled the creation of new mappings with all matchers
  • Enabled the creation of a mapping with a body
  • Enabled basic JSON body validation (XML body validation pending)
  • Enabled persistence flag for mappings
  • Enabled preloading mock servers by editing a configuration file
  • Created proxyBaseUrl functionality
  • Creating a new section for Scenario definitions for mappings

@El-kady
Copy link

El-kady commented Feb 14, 2023

why this request did not merged?

@kunalashar25
Copy link

kunalashar25 commented Apr 23, 2025

@antoni-g I tried running this project with your changes, I am able to launch it locally but unable to create any mapping. Upon clicking the save button, the app freezes and I can't see any request being made in the Network tab (no console errors as well).

There is one error in the project Property 'mappings' does not exist on type 'unknown'.

MapSubscriber.project
src/modules/mappings/store/epics.ts:53
  50 | .pipe(
  51 |     mergeMap(({ payload }: ILoadServerMappingsRequestAction) =>
  52 |         getMappings(payload.server).pipe(
> 53 |             map(({ mappings }) => loadServerMappingsSuccess(
  54 |                 payload.server,
  55 |                 mappings
  56 |             ))

Can you please help to check once? Let me know if you need any further details from my end.

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