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

Ensure new claims do not send non-standard inputs #390

Open
YashKumarVerma opened this issue May 30, 2020 · 1 comment · May be fixed by #391
Open

Ensure new claims do not send non-standard inputs #390

YashKumarVerma opened this issue May 30, 2020 · 1 comment · May be fixed by #391

Comments

@YashKumarVerma
Copy link
Contributor

@YashKumarVerma can you first create a PR with the following:

  • in add claim api, clean the urls, removing all the hashes, query params and add trailing slash.
  • write a script to transform the current items for the same.

Originally posted by /hereisnaman in #279 (comment)
@boss-contributions-bot
Copy link

Thanks @YashKumarVerma, for raising the issue! 🙌

One of our mentors will revert on this soon. ✅

Star ⭐ this project and tweet 🐦 about BOSS 2020.

YashKumarVerma added a commit to YashKumarVerma/boss that referenced this issue May 30, 2020
@YashKumarVerma
Ensure new claims have clean urls, Fixes: coding-blocks#390
395403d
YashKumarVerma added a commit to YashKumarVerma/boss that referenced this issue May 30, 2020
@YashKumarVerma
Fix old url issues, Ref coding-blocks#390
c31dbbf
@YashKumarVerma YashKumarVerma linked a pull request May 30, 2020 that will close this issue
Open
YashKumarVerma added a commit to YashKumarVerma/boss that referenced this issue May 30, 2020
@YashKumarVerma
Fix old url issues, Ref coding-blocks#390
1236509
This was referenced May 30, 2020
Open
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Clean claim inputs YashKumarVerma/boss
1 participant
@YashKumarVerma