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

CLI: Can't invite users #614

Closed
rajdip-b opened this issue Jan 9, 2025 · 9 comments · Fixed by #648
Closed

CLI: Can't invite users #614

rajdip-b opened this issue Jan 9, 2025 · 9 comments · Fixed by #648
Assignees
@csehatt741
Labels
difficulty: 3 good first issue Good for newcomers help wanted Extra attention is needed priority: high released on @develop scope: cli Everything related to the CLI type: bug Something isn't working

Comments

@rajdip-b
Copy link
Member

rajdip-b commented Jan 9, 2025

Describe the bug

When using keyshade workspace invite <workspace-slug> -e <user-email> -r <role>, the API call fails.
image
@rajdip-b rajdip-b added type: bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed priority: high difficulty: 3 scope: cli Everything related to the CLI labels Jan 9, 2025
@bansal-harsh-2504
Copy link
Contributor

/attempt

@github-actions GitHub Actions
Copy link

Assigned the issue to @bansal-harsh-2504!

@bansal-harsh-2504
Copy link
Contributor

everytime i run

pnpm dev:cli

i get

secp256k1 unavailable, reverting to browser version
@keyshade/cli:dev: Usage: index [options] [command]
@keyshade/cli:dev:
@keyshade/cli:dev: Options:
@keyshade/cli:dev:   --profile <string>   The profile to use
@keyshade/cli:dev:   --api-key <string>   The API key to use
@keyshade/cli:dev:   --base-url <string>  The base URL to use
@keyshade/cli:dev:   -h, --help           display help for command
@keyshade/cli:dev:
@keyshade/cli:dev: Commands:
@keyshade/cli:dev:   run <command>        Run a command
@keyshade/cli:dev:   init [options]       Initialize the configurations to enable live-updates in
@keyshade/cli:dev:                        the current project
@keyshade/cli:dev:   profile              Manage your CLI profiles
@keyshade/cli:dev:   workspace            Manages the workspaces on keyshade
@keyshade/cli:dev:   project              Manage projects of a workspace
@keyshade/cli:dev:   environment          Manage your environments in keyshade.
@keyshade/cli:dev:   secret               Manages the secrets on keyshade
@keyshade/cli:dev:   scan [options]       Scan the project to detect any hardcoded secrets
@keyshade/cli:dev:   variable             Manages the variable on keyshade
@keyshade/cli:dev:   help [command]       display help for command
@keyshade/cli:dev:  ELIFECYCLE  Command failed with exit code 1.
@keyshade/cli:dev: ERROR: command finished with error: command (D:\keyshade\apps\cli) C:\Users\ASUS\AppData\Roaming\npm\pnpm.cmd run dev exited (1)
@keyshade/cli#dev: command (D:\keyshade\apps\cli) C:\Users\ASUS\AppData\Roaming\npm\pnpm.cmd run dev exited (1)

 Tasks:    0 successful, 1 total
Cached:    0 cached, 1 total
  Time:    3.85s
Failed:    @keyshade/cli#dev

Do I need to populate any env variables to recreate this error and run cli successfully?

@rajdip-b

@rajdip-b
Copy link
Member Author

CLI wont require env vars, but you do need to create your account on stage.api.keyshade.xyz. Refer to bruno for that.

  • Create a profile
  • Update your profile and set isOnboardingFinished to true
  • Create an API key and add ADMIN in the authorities array
  • Use keyshade profile create --name <> --base-url <> --api-key <> command to initialize the CLI

@bansal-harsh-2504 bansal-harsh-2504 removed their assignment Jan 14, 2025
@csehatt741
Copy link
Contributor

/attempt

@github-actions GitHub Actions
Copy link

Assigned the issue to @csehatt741!

@csehatt741
Copy link
Contributor

@rajdip-b, I could not reproduce this issue, as invitation succeeds:

Image
Image

Could you clarify the issue and refine the description, please?

@rajdip-b
Copy link
Member Author

Image Image

The invitation goes through (as we can see), but then, we can see that the entry isn't present in WorkspaceMember table. There should be an entry with invitationAccepted set to true

csehatt741 added a commit to csehatt741/keyshade that referenced this issue Jan 20, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (keyshade-xyz…
4b50f11 
…#614)
@csehatt741 csehatt741 mentioned this issue Jan 20, 2025
Merged
10 tasks
csehatt741 added a commit to csehatt741/keyshade that referenced this issue Jan 21, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (keyshade-xyz…
4e8ee18 
…#614)
csehatt741 added a commit to csehatt741/keyshade that referenced this issue Jan 24, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (keyshade-xyz…
4bbdfcf 
…#614)
csehatt741 added a commit to csehatt741/keyshade that referenced this issue Jan 24, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (keyshade-xyz…
906ea67 
…#614)
csehatt741 added a commit to csehatt741/keyshade that referenced this issue Jan 25, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (keyshade-xyz…
f3f23c5 
…#614)
rajdip-b pushed a commit that referenced this issue Jan 25, 2025
@csehatt741
fix(cli): Workspace membership API client payload fixed (#614) (#648)
e23057b
rajdip-b pushed a commit that referenced this issue Jan 25, 2025
@semantic-release-bot
chore(release): 2.10.0-stage.4 [skip ci]
ae7c44f 
## [2.10.0-stage.4](v2.10.0-stage.3...v2.10.0-stage.4) (2025-01-25)

### 🐛 Bug Fixes

* **cli:** Workspace membership API client payload fixed ([#614](#614)) ([#648](#648)) ([e23057b](e23057b))
@rajdip-b
Copy link
Member Author

🎉 This issue has been resolved in version 2.10.0-stage.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@csehatt741 csehatt741

Labels
difficulty: 3 good first issue Good for newcomers help wanted Extra attention is needed priority: high released on @develop scope: cli Everything related to the CLI type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(cli): Workspace membership API client payload fixed (#614) csehatt741/keyshade
3 participants
@csehatt741 @rajdip-b @bansal-harsh-2504