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.

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

refactor(api): Update return type while fetching secrets and variables #264

Merged
merged 4 commits into from
Jun 9, 2024

Conversation

Ritika1705
Copy link
Contributor

@Ritika1705 Ritika1705 commented Jun 8, 2024

Description

Updated return type while fetching secrets and variables

Fixes #261

Dependencies

Mention any dependencies/packages used

Future Improvements

Mention any improvements to be done in future related to any file/feature

Mentions

Mention and tag the people

Screenshots of relevant screens

Add screenshots of relevant screens

Developer's checklist

  • My PR follows the style guidelines of this project
  • I have performed a self-check on my work

If changes are made in the code:

  • I have followed the coding guidelines
  • My changes in code generate no new warnings
  • My changes are breaking another fix/feature of the project
  • I have added test cases to show that my feature works
  • I have added relevant screenshots in my PR
  • There are no UI/UX issues

Documentation Update

  • This PR requires an update to the documentation at docs.keyshade.xyz
  • I have made the necessary updates to the documentation, or no documentation changes are required.

@Ritika1705 Ritika1705 requested a review from rajdip-b as a code owner June 8, 2024 11:35
Copy link
Member

@rajdip-b rajdip-b left a comment

Choose a reason for hiding this comment

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

Additionally, you would need to add tests for variable service aswell!

apps/api/src/secret/secret.e2e.spec.ts Outdated Show resolved Hide resolved
apps/api/src/secret/service/secret.service.ts Outdated Show resolved Hide resolved
apps/api/src/variable/service/variable.service.ts Outdated Show resolved Hide resolved
apps/api/src/variable/service/variable.service.ts Outdated Show resolved Hide resolved
apps/api/src/variable/service/variable.service.ts Outdated Show resolved Hide resolved
@rajdip-b rajdip-b changed the title Update return type while fetching secrets and variables refactor(api): Update return type while fetching secrets and variables Jun 9, 2024
@Ritika1705
Copy link
Contributor Author

Format

image

apps/api/src/secret/secret.types.ts Outdated Show resolved Hide resolved
apps/api/src/secret/service/secret.service.ts Outdated Show resolved Hide resolved
apps/api/src/variable/variable.e2e.spec.ts Outdated Show resolved Hide resolved
apps/api/src/variable/variable.types.ts Outdated Show resolved Hide resolved
apps/api/src/secret/service/secret.service.ts Outdated Show resolved Hide resolved
apps/api/src/secret/secret.e2e.spec.ts Outdated Show resolved Hide resolved
@rajdip-b
Copy link
Member

rajdip-b commented Jun 9, 2024

Format

image

This looks perfect!

apps/api/src/secret/service/secret.service.ts Outdated Show resolved Hide resolved
apps/api/src/secret/service/secret.service.ts Outdated Show resolved Hide resolved
@Ritika1705
Copy link
Contributor Author

@rajdip-b , done!!

Copy link

sonarcloud bot commented Jun 9, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.9% Duplication on New Code

See analysis details on SonarCloud

Copy link

codecov bot commented Jun 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.65%. Comparing base (ce50743) to head (3de8cd6).
Report is 26 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #264      +/-   ##
===========================================
- Coverage    91.71%   90.65%   -1.06%     
===========================================
  Files          111      113       +2     
  Lines         2510     2750     +240     
  Branches       469      526      +57     
===========================================
+ Hits          2302     2493     +191     
- Misses         208      257      +49     
Flag Coverage Δ
api-e2e-tests 90.65% <100.00%> (-1.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rajdip-b rajdip-b merged commit fd36abd into keyshade-xyz:develop Jun 9, 2024
7 checks passed
rajdip-b pushed a commit that referenced this pull request Jun 12, 2024
## [2.0.0](v1.4.0...v2.0.0) (2024-06-12)

### ⚠ BREAKING CHANGES

* **api:** Refactor environment, [secure] and variable functionality

### 🚀 Features

* **platform:** Workspace integrate ([#241](#241)) ([6107e7d](6107e7d))

### 📚 Documentation

* Fix broken links in README.md ([9266788](9266788))
* Modified environment-variable.md ([#256](#256)) ([4974756](4974756))

### 🔧 Miscellaneous Chores

* Added docker build and run commands to` package.json` ([#258](#258)) ([af61791](af61791))
* **api:** Fix inconsistencies in zod schema ([#240](#240)) ([f3a3632](f3a3632))
* **ci:** Update deploy web ([e80d47d](e80d47d))
* **docker:** Grant correct permissions to docker image ([#251](#251)) ([49546aa](49546aa))
* Update GitHub Action plugin versions  ([#263](#263)) ([020bbf6](020bbf6))
* Update package versions for release ([93785be](93785be))

### 🔨 Code Refactoring

* **api:** Refactor environment, [secure] and variable functionality ([#270](#270)) ([55a6d37](55a6d37))
* **api:** Replace for loop with array indexing while decrypting [secure]s during bulk fetch [#265](#265) ([#266](#266)) ([62a1731](62a1731))
* **api:** Update return type while fetching [secure]s and variables ([#264](#264)) ([fd36abd](fd36abd))
@rajdip-b
Copy link
Member

🎉 This PR is included in version 2.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

rajdip-b pushed a commit that referenced this pull request Jun 27, 2024
## [2.0.0](v1.4.0...v2.0.0) (2024-06-12)

### ⚠ BREAKING CHANGES

* **api:** Refactor environment, [secure] and variable functionality

### 🚀 Features

* **platform:** Workspace integrate ([#241](#241)) ([6107e7d](6107e7d))

### 📚 Documentation

* Fix broken links in README.md ([9266788](9266788))
* Modified environment-variable.md ([#256](#256)) ([4974756](4974756))

### 🔧 Miscellaneous Chores

* Added docker build and run commands to` package.json` ([#258](#258)) ([af61791](af61791))
* **api:** Fix inconsistencies in zod schema ([#240](#240)) ([f3a3632](f3a3632))
* **ci:** Update deploy web ([e80d47d](e80d47d))
* **docker:** Grant correct permissions to docker image ([#251](#251)) ([49546aa](49546aa))
* Update GitHub Action plugin versions  ([#263](#263)) ([020bbf6](020bbf6))
* Update package versions for release ([93785be](93785be))

### 🔨 Code Refactoring

* **api:** Refactor environment, [secure] and variable functionality ([#270](#270)) ([55a6d37](55a6d37))
* **api:** Replace for loop with array indexing while decrypting [secure]s during bulk fetch [#265](#265) ([#266](#266)) ([62a1731](62a1731))
* **api:** Update return type while fetching [secure]s and variables ([#264](#264)) ([fd36abd](fd36abd))
yogesh1801 pushed a commit to yogesh1801/keyshade that referenced this pull request Jun 29, 2024
## [2.0.0](keyshade-xyz/keyshade@v1.4.0...v2.0.0) (2024-06-12)

### ⚠ BREAKING CHANGES

* **api:** Refactor environment, [secure] and variable functionality

### 🚀 Features

* **platform:** Workspace integrate ([keyshade-xyz#241](keyshade-xyz#241)) ([6107e7d](keyshade-xyz@6107e7d))

### 📚 Documentation

* Fix broken links in README.md ([9266788](keyshade-xyz@9266788))
* Modified environment-variable.md ([keyshade-xyz#256](keyshade-xyz#256)) ([4974756](keyshade-xyz@4974756))

### 🔧 Miscellaneous Chores

* Added docker build and run commands to` package.json` ([keyshade-xyz#258](keyshade-xyz#258)) ([af61791](keyshade-xyz@af61791))
* **api:** Fix inconsistencies in zod schema ([keyshade-xyz#240](keyshade-xyz#240)) ([f3a3632](keyshade-xyz@f3a3632))
* **ci:** Update deploy web ([e80d47d](keyshade-xyz@e80d47d))
* **docker:** Grant correct permissions to docker image ([keyshade-xyz#251](keyshade-xyz#251)) ([49546aa](keyshade-xyz@49546aa))
* Update GitHub Action plugin versions  ([keyshade-xyz#263](keyshade-xyz#263)) ([020bbf6](keyshade-xyz@020bbf6))
* Update package versions for release ([93785be](keyshade-xyz@93785be))

### 🔨 Code Refactoring

* **api:** Refactor environment, [secure] and variable functionality ([keyshade-xyz#270](keyshade-xyz#270)) ([55a6d37](keyshade-xyz@55a6d37))
* **api:** Replace for loop with array indexing while decrypting [secure]s during bulk fetch [keyshade-xyz#265](keyshade-xyz#265) ([keyshade-xyz#266](keyshade-xyz#266)) ([62a1731](keyshade-xyz@62a1731))
* **api:** Update return type while fetching [secure]s and variables ([keyshade-xyz#264](keyshade-xyz#264)) ([fd36abd](keyshade-xyz@fd36abd))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API: Update return type while fetching secrets and variables
2 participants