-
-
Notifications
You must be signed in to change notification settings - Fork 162
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
BUG: Secrets get created even with empty names #701
Comments
can you assign me this ? |
Hey @Agastya221, @csehatt741 would be working on the backend side of things. If you would like to attempt it, then you can get started with the frontend changes. |
@csehatt741 could you please make your attempt? |
/attempt |
Assigned the issue to @csehatt741! |
@rajdip-b yes sure assign me the frontend part |
Great! |
Just do keep in mind that this issue belongs to FOSS Hack 2025. Do read their guidelines, register yourself and submit the PR during the hackathon window. |
/attempt |
Assigned the issue to @Sohamnimbalkar07! |
Hey @Sohamnimbalkar07, I would have to unassign you since there are 2 contributors already attempting this issue. Please refrain from attempting issues that already have assignees. |
Sure @rajdip-b. Thanks for letting me know! |
I`ve created a draft PR including only backend fixes. |
@csehatt741 Frontend is already done but since it's FossHacks issue We need to pr when the event starts |
Actually, that is another reason I created a draft PR, so you can also commit your changes into the same branch and have the PR ready only when FossHacks is opened. |
Nah, drafts aren't allowed either. Yes you can save the changes in a branch, and save them for later for sure. |
Though, I`m not much interested in FOSSHack, but if it is important we can close this PR and create a new one later. |
I`ve finished working on the backend and can be unassigned from this issue. |
Describe the bug
We are able to create secrets with empty names in the platform.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Secrets shouldn't be created with empty names
Screenshots
Additional Context
The text was updated successfully, but these errors were encountered: