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

fix sanitization for 1353 #1528

Merged
merged 5 commits into from
Oct 31, 2023
Merged

fix sanitization for 1353 #1528

merged 5 commits into from
Oct 31, 2023

Conversation

manthan-sharma-23
Copy link
Contributor

#1353 fix

@JisanAR03
Copy link
Contributor

can you please stop work on bootstrap.js? I already fixed those and am just waiting to confirm other PR then i will make the pr again (I didn't make those PR because of conflict issue),

@manthan-sharma-23
Copy link
Contributor Author

Yeah sure mate ! ,didn't know that.

@JisanAR03
Copy link
Contributor

Yeah sure mate ! ,didn't know that.

@manthan-sharma-23 thank you so much😊

@JisanAR03
Copy link
Contributor

the code should like :
image
because title could be anything and we have to apply condition to them ,, I already solve this issue just didn't make pull request because of my some PR are pending and this update is depend on pending PR

@DonnieBLT DonnieBLT enabled auto-merge (rebase) October 31, 2023 03:35
@DonnieBLT DonnieBLT merged commit 4500aa9 into OWASP-BLT:main Oct 31, 2023
3 checks passed
@JisanAR03 JisanAR03 mentioned this pull request Oct 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@manthan-sharma-23 @JisanAR03 @DonnieBLT