Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ZMS-101 #553

Merged
merged 34 commits into from
Nov 30, 2023
Merged

ZMS-101 #553

merged 34 commits into from
Nov 30, 2023

Conversation

NickOvt
Copy link
Contributor

@NickOvt NickOvt commented Nov 10, 2023

  1. Description now stays where it was defined, that is if using the same component the fields will still have the same component, but different descriptions
  2. Added missing descriptions for the fields in the schemas in the messages-schemas.js file

@NickOvt
api.js added endpoint for generating openapi docs. added new info to …
e4c30ca 
…one route in mailboxes.js and messages.js files so that the api docs generation can be done at all
@NickOvt
try to first generate json representation of the api docs
d2caad5
@NickOvt
add initial Joi Object parsing
a69c6ad
@NickOvt
api.js make generation dynamic. messages.js add schemas from separate…
e7a3c72 
… file. messages-schemas.js used for messages endpoint schemas
@NickOvt
add additions to schemas. Add new schemas to messages.js and also add…
f3fe6cb 
… response object there. Add response object parsing functionality to api.js
@NickOvt
add initial openapi doc yml file generation
cccd2c0
@NickOvt
remove manual yaml parsing with js-yaml JSON -> YAML parsing
c58a969
@NickOvt
fix replaceWithRefs and parseComponentsDecoupled functions, refactor,…
d57405c 
… remove unnecessary comments and logs
@NickOvt
add support for another endpoint
e5e72cb
@NickOvt
move big code from api.js to tools
09632a5
@NickOvt
fix array type representation, fix response objects, add necessary da…
dc3e7ee 
…ta and changes to endpoints
@NickOvt
Merge branch 'master' into ZMS-87
35f19d5
@NickOvt
redo include logic into exclude login
9ba060c
@NickOvt
Merge branch 'ZMS-87' of github.com:nodemailer/wildduck into ZMS-87
77bbe9e
@NickOvt
fix api generation in tools.js to accomodate new naming of objects
d6aae5f
@NickOvt
fix messages.js, add structuredClone check in tools.js
06b9a45
@NickOvt
fix structured clone definition
419e3ec
@NickOvt
Merge branch 'master' of github.com:nodemailer/wildduck into ZMS-87
740ae00
@NickOvt
add one endpoint in messages.js to the api generation
b5df32a
@NickOvt
messages.js add one more endpoint to API generation
e8513d4
@NickOvt
add response to prev commit. Add new endpoint to API generation. Arch…
73b96e0 
…ive message and archive messages
@NickOvt
finish with post endpoints in messages.js
417c093
@NickOvt
added general request and response schemas. Also added req and res sc…
1db7528 
…hemas for messages
@NickOvt
add multiple GET endpoints to API generation and changed them to new …
71c6496 
…design. Use general schemas made earlier
@NickOvt
fix incorrect import of successRes
00a5afb
@NickOvt
fix mailboxes.js
fd310ae
@NickOvt
refactor general-schemas.js. Fix searchSchema in messages.js. Mailbox…
2d31d2b 
…es.js fix response
@NickOvt
tools.js rename methodObj in API generation to operationObj
5e4a20a
@NickOvt
tools.js api generation remove string fallbacks
9234b0d
@NickOvt
messages.js finish with GET endpoints, addition to API doc generation
70ebdbc
@NickOvt NickOvt changed the base branch from ZMS-87 to master November 10, 2023 09:08
@NickOvt NickOvt changed the title Zms 101 ZMS-101 Nov 10, 2023
@NickOvt NickOvt requested a review from andris9 November 10, 2023 09:24
andris9
andris9 approved these changes Nov 10, 2023
View reviewed changes
Copy link
Member

@andris9 andris9 left a comment

Choose a reason for hiding this comment

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

LGTM

@andris9
Copy link
Member

andris9 commented Nov 10, 2023

Conflicts with previous PR. Can not merge.

@andris9 andris9 merged commit 3c9e175 into master Nov 30, 2023
@andris9 andris9 deleted the ZMS-101 branch November 30, 2023 12:27
@github-actions github-actions bot mentioned this pull request Nov 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andris9 andris9 andris9 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NickOvt @andris9