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

Able to create events with white spaces only. #1021

Closed
skbhagat0502 opened this issue Oct 31, 2023 · 6 comments · Fixed by #1025
Closed

Able to create events with white spaces only. #1021

skbhagat0502 opened this issue Oct 31, 2023 · 6 comments · Fixed by #1025
Assignees
Labels
bug Something isn't working feature request

Comments

@skbhagat0502
Copy link
Contributor

Describe the bug
When attempting to create an event using the event creation modal, it is possible to submit the form with no title, no description, and no location, by simply inputting spaces.

To Reproduce
Steps to reproduce the behavior:

  1. Login to talawa admin.
  2. Go to the dashboard and then events.
  3. Click on "Add event" button.
  4. Enter spaces in the title field.
  5. Enter spaces in the description field.
  6. Enter spaces in the location field.
  7. Attempt to create the event with the filled but empty information.

Expected behavior
I expected the system to validate and prevent the creation of an event with no meaningful information, ensuring at least one of the following fields (title, description, location) is non-empty.

Actual behavior
The system allows the creation of an event even when all fields (title, description, location) contain only white spaces.

Screenshots

2023-11-01.01-19-28.mp4

Additional details
Add any other context or screenshots about the feature request here.

@skbhagat0502 skbhagat0502 added the bug Something isn't working label Oct 31, 2023
@skbhagat0502
Copy link
Contributor Author

@palisadoes Please assign me this issue.

@skbhagat0502 skbhagat0502 changed the title Events can be created with blac Able to create events with white spaces only. Oct 31, 2023
@palisadoes
Copy link
Contributor

@EshaanAgg Any comments on this?

@akhilender-bongirwar
Copy link
Contributor

Hey @skbhagat0502, since you already have 2 issues assigned to you, may I please take up this issue and resolve it?
// @palisadoes

@skbhagat0502
Copy link
Contributor Author

Hey @skbhagat0502, since you already have 2 issues assigned to you, may I please take up this issue and resolve it? // @palisadoes

Thanks for your interest, @akhilender-bongirwar , I appreciate your willingness to help out. However, I've already started working on this issue and I'm confident in my ability to address it effectively. If you have any other tasks or issues you'd like to tackle, feel free to let me know, and we can coordinate efforts.

@palisadoes
Copy link
Contributor

@skbhagat0502 We will soon start enforcing the two issue guidance.

@skbhagat0502
Copy link
Contributor Author

Ok @palisadoes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants