Apidoc#21
Conversation
api-doc/api-doc.yaml
Outdated
| application.json: | ||
| schema: | ||
| "$ref": "#/components/schemas/user/acceptTermsAndConditionResponse400" | ||
| "/user/v1/account/verifyMentor": |
There was a problem hiding this comment.
Please check this API response
api-doc/api-doc.yaml
Outdated
| application.json: | ||
| schema: | ||
| "$ref": "#/components/schemas/form/readForm400Response" | ||
| "/user/v1/mentors/list": |
There was a problem hiding this comment.
@ankitpws pageNo,pageSize and search text is missing
And in other API's as well if required
api-doc/api-doc.yaml
Outdated
| content: | ||
| application.json: | ||
| schema: | ||
| "$ref": "#/components/schemas/mentor/getMentorList200Resposne" |
There was a problem hiding this comment.
@ankitpws Count filed is missing in the response
There was a problem hiding this comment.
added count filed
api-doc/api-doc.yaml
Outdated
| application.json: | ||
| schema: | ||
| "$ref": "#/components/schemas/mentor/getMentorList200Resposne" | ||
| "/user/v1/userentity/create": |
There was a problem hiding this comment.
@ankitpws Validate in swagger editor, this API not showing in swagger editor list
api-doc/temp.json
Outdated
| { | ||
| "openapi": "3.0.0", | ||
| "info": { | ||
| "title": "ML-Core Service API(s)", |
There was a problem hiding this comment.
@ankitpws this file is not required
There was a problem hiding this comment.
yes sorry i forgot to delete
app.js
Outdated
| specUrl: "/docs/swagger", | ||
| }) | ||
| ); | ||
| // app.use("/api-docs", swaggerUi.serve); |
There was a problem hiding this comment.
@ankitpws remove unnecessary comments
configs/mongodb.js
Outdated
| const mongoose_autopopulate = require("mongoose-autopopulate"); | ||
| const mongoose_timestamp = require("mongoose-timestamp"); | ||
| const mongoose_paginate = require('mongoose-paginate-v2'); | ||
| const mongoose_paginate = require("mongoose-paginate-v2"); |
There was a problem hiding this comment.
@ankitpws if not require remove mongoose_paginate
controllers/v1/profile.js
Outdated
| async update(req) { | ||
| const params = req.body; | ||
| try { | ||
| const updatedProfile = await profileHelper.update( |
There was a problem hiding this comment.
@ankitpws please verify they should not be able to update the email
There was a problem hiding this comment.
its done please check service file for update method.
There was a problem hiding this comment.
already resolved
controllers/v1/mentors.js
Outdated
| async list(req) { | ||
| const userId = req.decodedToken._id; | ||
| try { | ||
| console.log("=======>", req.pageNo, req.pageSize, req.searchText, userId); |
There was a problem hiding this comment.
@ankitpws remove the console.log
routes/index.js
Outdated
| let validationError; | ||
|
|
||
| console.log(".//////////////", req.method); | ||
| console.log(".//////////////", req.pageNo, req.pageSize, req.searchText); |
There was a problem hiding this comment.
@ankitpws - remove this console.log
src/app.js
Outdated
| app.use(expressFileUpload()) | ||
|
|
||
| app.get("/api-doc", function (req, res) { | ||
| console.log("api"); |
There was a problem hiding this comment.
@ankitpws remove this console.log
|
Kudos, SonarCloud Quality Gate passed!
|
2 participants
I have added Api doc and updated packages for users.