Skip to content

feat(openapi-sender): migrate sender to a dedicated package MAPCO-9760#213

Open
netanelC wants to merge 6 commits into
create-handlerfrom
create-sender
Open

feat(openapi-sender): migrate sender to a dedicated package MAPCO-9760#213
netanelC wants to merge 6 commits into
create-handlerfrom
create-sender

Conversation

@netanelC

@netanelC netanelC commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@netanelC netanelC changed the title feat(openapi-sender): create sender feat(openapi-sender): migrate handler to a dedicated package MAPCO-9760 Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

🎫 Related Jira Issue: MAPCO-9760

@netanelC netanelC changed the title feat(openapi-sender): migrate handler to a dedicated package MAPCO-9760 feat(openapi-sender): migrate sender to a dedicated package MAPCO-9760 Jun 9, 2026

@ronenkapelian ronenkapelian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  1. .vscode/project.code-workspace — Add folder entry
{
  "name": "openapi-typed-request-handler",
  "path": "../packages/openapi-typed-request-sender",
},
  1. add knip to package.json and validate it
"knip": "knip --directory ../.. --workspace packages/openapi-typed-request-sender"
  1. add typedocs

  2. fix api-extractor:
    a. add etc dir under "packages/openapi-typed-request-sender"
    b.

Image

then refactor this lines on index.ts
c. run pnpm run api and add the output file as part of the commit
d. it will solve the ci problem

image

validate ci after it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image

update the example

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