-
-
Notifications
You must be signed in to change notification settings - Fork 105
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
chore: Remove swagger docs #220
chore: Remove swagger docs #220
Conversation
PR Description updated to latest commit (1e9d5bb) |
PR Review 🔍
|
PR Code Suggestions ✨No code suggestions found for PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks clean, although there are a few unused imports:
- https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=220&id=keyshade-xyz_keyshade&open=AY98D89neI_RmlPjPfjM
- https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=220&id=keyshade-xyz_keyshade&open=AY98D88WeI_RmlPjPfjL
We will be good to go once you remove these!
…yshade into chore/remove-swagger-docs
Quality Gate passedIssues Measures |
## [1.4.0](v1.3.0...v1.4.0) (2024-05-24) ### 🚀 Features * add example for health and email auth ([b834d25](b834d25)) * **api:** Add `minio-client` provider ([#237](#237)) ([cd71c5a](cd71c5a)) * **api:** Add feature to fork projects ([#239](#239)) ([3bab653](3bab653)) * **api:** Added feedback form module ([#210](#210)) ([ae1efd8](ae1efd8)) * **api:** Added Project Level Access ([#221](#221)) ([564f5ed](564f5ed)) * **api:** Added support for changing email of users ([#233](#233)) ([5ea9a10](5ea9a10)) * implemented auth, ui for most, and fixed cors ([#217](#217)) ([feace86](feace86)) * **platfrom:** add delete method in api client ([#225](#225)) ([55cf09f](55cf09f)) * **postman:** add example for get_self and update_self ([e015acf](e015acf)) * **web:** Add and link privacy and tnc page ([#226](#226)) ([ec81eb9](ec81eb9)) ### 🐛 Bug Fixes * **web:** docker next config not found ([#228](#228)) ([afe3160](afe3160)) ### 📚 Documentation * Added docs regarding postman, and refactored architecture diagrams ([f1c9777](f1c9777)) * Fix typo in organization-of-code.md ([#234](#234)) ([11244a2](11244a2)) ### 🔧 Miscellaneous Chores * **api:** Get feedback forward email from process.env ([#236](#236)) ([204c9d1](204c9d1)) * **postman:** Initialized postman ([bb76384](bb76384)) * **release:** Update changelog config ([af91283](af91283)) * Remove swagger docs ([#220](#220)) ([7640299](7640299)) ### 🔨 Code Refactoring * **api:** Replaced OTP code from alphanumeric to numeric ([#230](#230)) ([f16162a](f16162a))
🎉 This PR is included in version 1.4.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
User description
Description
main.ts
.Fixes #218
Mentions
@rajdip-b
Screenshots of relevant screens
Add screenshots of relevant screens
Developer's checklist
If changes are made in the code:
Documentation Update
PR Type
enhancement, documentation
Description
ApiTags
,ApiOperation
, andApiResponse
.Changes walkthrough 📝
12 files
api-key.controller.ts
Remove Swagger Documentation from API Key Controller
apps/api/src/api-key/controller/api-key.controller.ts
documentation.
create.api-key.ts
Simplify CreateApiKey DTO by Removing Swagger Decorators
apps/api/src/api-key/dto/create.api-key/create.api-key.ts
ApiProperty
decorators from the DTO properties.app.controller.ts
Remove Swagger Tags from App Controller
apps/api/src/app/app.controller.ts
ApiTags
decorator from the App Controller.auth.controller.ts
Clean Up Auth Controller by Removing Swagger Documentation
apps/api/src/auth/controller/auth.controller.ts
methods.
environment.controller.ts
Remove Swagger Tags from Environment Controller
apps/api/src/environment/controller/environment.controller.ts
ApiTags
decorator from the Environment Controller.event.controller.ts
Remove Swagger Tags from Event Controller
apps/api/src/event/controller/event.controller.ts
ApiTags
decorator from the Event Controller.feedback.controller.ts
Remove Swagger Documentation from Feedback Controller
apps/api/src/feedback/controller/feedback.controller.ts
integration.controller.ts
Remove Swagger Tags from Integration Controller
apps/api/src/integration/controller/integration.controller.ts
ApiTags
decorator from the Integration Controller.main.ts
Remove Swagger Setup from Main Application File
apps/api/src/main.ts
file.
project.controller.ts
Remove Swagger Tags from Project Controller
apps/api/src/project/controller/project.controller.ts
ApiTags
decorator from the Project Controller.secret.controller.ts
Remove Swagger Tags from Secret Controller
apps/api/src/secret/controller/secret.controller.ts
ApiTags
decorator from the Secret Controller.variable.controller.ts
Remove Swagger Tags from Variable Controller
apps/api/src/variable/controller/variable.controller.ts
ApiTags
decorator from the Variable Controller.