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

[ Bug ] Validation needed in adding News #343

Closed
ishan0805 opened this issue Mar 16, 2021 · 4 comments
Closed

[ Bug ] Validation needed in adding News #343

ishan0805 opened this issue Mar 16, 2021 · 4 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ishan0805
Copy link
Contributor

Description

While adding a news two fields are required , but validation is not given to any of them . Thus a user can post a news but it will not display .

WhatsApp.Video.2021-03-16.at.12.17.10.PM.mp4

Additional Validation

Validation to the length of Title and description can also be given . As of now any long text can be added . This can create issue of overflow to data and the database can be easily be spammed with long texts .

Solution

Simple form validation can be added to solve this issue

@ram2939
Copy link

ram2939 commented Mar 16, 2021

I would like to work on this issue

@ishan0805
Copy link
Contributor Author

@ram2939 sorry I have already asked the mentor to allot this to me .

@palisadoes palisadoes added the unapproved Unapproved, needs to be triaged label Mar 16, 2021
@jordanliu jordanliu removed the unapproved Unapproved, needs to be triaged label Mar 17, 2021
@Sagar2366 Sagar2366 added bug Something isn't working good first issue Good for newcomers and removed enhancement labels Mar 17, 2021
@shauravshikhar
Copy link
Contributor

is this problem solved?

@ishan0805
Copy link
Contributor Author

Yes the PR is already generated you can review it 😊

Ayush0Chaudhary pushed a commit to Ayush0Chaudhary/talawa that referenced this issue Mar 29, 2023
* Plugin Schema Updates

* Plugin Added

* Creator Check Added

* Event Updates

* registerEvent and unregister event mutation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants