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

CLI: Add functionality to operate on Environments #300

Closed
rajdip-b opened this issue Jun 27, 2024 · 6 comments · Fixed by #324
Closed

CLI: Add functionality to operate on Environments #300

rajdip-b opened this issue Jun 27, 2024 · 6 comments · Fixed by #324
Assignees
Labels
difficulty: 3 priority: high scope: cli Everything related to the CLI type: feature Release drafter tag for tagging PRs related to adding new features

Comments

@rajdip-b
Copy link
Member

Description

We would like to perform the following operations for project using our CLI:

  • List all environments under a project (keyshade environment list <projectId>)
  • Get a particular environment by id (keyshade environment get <environmentId>)
  • Create an environment (keyshade environment create <projectId> (refer postman for parameters))
  • Edit an environment (keyshade environment update <environmentId> (refer postman for parameters))
  • Delete an environment (keyshade environment delete <environmentId>)

Solution

  • Create an implementation of command.interface.ts and name if EnvironmentCommand.
  • Stash all the functions in src/commands/environment.
  • Add environment specific API functionality in src/http/project.ts and name the class EnvironmentController.
@rajdip-b rajdip-b added type: feature Release drafter tag for tagging PRs related to adding new features priority: high foss hack Clustering all the curated issues for Foss Hack 2024 difficulty: 3 scope: cli Everything related to the CLI labels Jun 27, 2024
@vr-varad
Copy link
Contributor

vr-varad commented Jul 2, 2024

@rajdip-b As my last pr is approved I would like to work on this issue.

@rajdip-b
Copy link
Member Author

rajdip-b commented Jul 2, 2024

Hey! Sure. But could you hold up for a day or two? We are making some changes in how the configuration works. After that we will be sharing the how-to guide for developing the CLI.

@vr-varad
Copy link
Contributor

vr-varad commented Jul 2, 2024

yeah Sure.

@rajdip-b rajdip-b removed the foss hack Clustering all the curated issues for Foss Hack 2024 label Jul 3, 2024
@rajdip-b
Copy link
Member Author

rajdip-b commented Jul 5, 2024

@vr-varad Hey bro! Could you make any progress on this yet?

@vr-varad
Copy link
Contributor

vr-varad commented Jul 5, 2024

Yeah Actually I was waiting for ur confirmation @rajdip-b Should I make a pr now?

@rajdip-b
Copy link
Member Author

rajdip-b commented Jul 5, 2024

Oh, yeah absolutely! You can make it now. This isn't related to foss hack so won't be an issue.

P.S. Absolutely sorry! I forgot to notify you in here. We maintain all of our announcements in Discord, and we did notify in there. So I assumed you were a community member :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: 3 priority: high scope: cli Everything related to the CLI type: feature Release drafter tag for tagging PRs related to adding new features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants