Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Sarthak5598
Copy link
Member

@Sarthak5598
Copy link
Member Author

image.png

image-1.png

@Sarthak5598 Sarthak5598 enabled auto-merge July 16, 2024 10:16
@Sarthak5598
Copy link
Member Author

BLT.-.Opera.2024-07-16.14-39-05.mp4

Copy link

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please look into the following comments when you get a chance:

@Sarthak5598
Copy link
Member Author

Thanks for the Review @arkid15r

@Sarthak5598 Sarthak5598 requested a review from arkid15r July 18, 2024 13:47
Copy link

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the code!

DonnieBLT
DonnieBLT previously approved these changes Jul 21, 2024
Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DonnieBLT
Copy link
Collaborator

please make sure all comments are resolved

Sarthak5598 and others added 4 commits July 26, 2024 23:53
Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
@Sarthak5598 Sarthak5598 requested a review from arkid15r July 27, 2024 10:08
@DonnieBLT
Copy link
Collaborator

Have all comments been resolved?

@Sarthak5598
Copy link
Member Author

yes everything is resolved @DonnieBLT

@Sarthak5598 Sarthak5598 disabled auto-merge July 28, 2024 06:52
@Sarthak5598 Sarthak5598 dismissed arkid15r’s stale review July 28, 2024 06:55

All reviews are resolved not sure why it says there are issues to resolve!

@Sarthak5598 Sarthak5598 merged commit 227ba57 into OWASP-BLT:main Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants