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

Skip to content

Create an Ideas & Suggestions page with voting #582 #2443

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
Merged
Sarthak5598 merged 28 commits into from
Jul 28, 2024
Merged

Create an Ideas & Suggestions page with voting #582 #2443

Sarthak5598 merged 28 commits into from
Jul 28, 2024

Conversation

@Sarthak5598
Copy link
Member

@Sarthak5598 Sarthak5598 commented Jul 14, 2024

@Sarthak5598
Copy link
Member Author

Sarthak5598 commented Jul 14, 2024

image.png

image-1.png

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

Sarthak5598 commented Jul 16, 2024

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

arkid15r
arkid15r previously requested changes Jul 18, 2024
View reviewed changes
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

Sarthak5598 commented Jul 18, 2024

Thanks for the Review @arkid15r

@Sarthak5598 Sarthak5598 requested a review from arkid15r July 18, 2024 13:47
@Sarthak5598 Sarthak5598 mentioned this pull request Jul 19, 2024
Closed
arkid15r
arkid15r reviewed Jul 19, 2024
View reviewed changes
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!

arkid15r
arkid15r reviewed Jul 19, 2024
View reviewed changes
DonnieBLT
DonnieBLT reviewed Jul 21, 2024
View reviewed changes
DonnieBLT
DonnieBLT previously approved these changes Jul 21, 2024
View reviewed changes
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

DonnieBLT commented Jul 21, 2024

please make sure all comments are resolved

arkid15r
arkid15r reviewed Jul 26, 2024
View reviewed changes
Sarthak5598 and others added 4 commits July 26, 2024 23:53
@Sarthak5598 @arkid15r
Update website/views.py
6a7ac94 
Co-authored-by: Arkadii Yakovets <[email protected]>
@Sarthak5598 @arkid15r
Update website/views.py
fe8a29a 
Co-authored-by: Arkadii Yakovets <[email protected]>
@Sarthak5598 @arkid15r
Update website/views.py
55bf55e 
Co-authored-by: Arkadii Yakovets <[email protected]>
@Sarthak5598
pre - commit
50a9295
@Sarthak5598 Sarthak5598 requested a review from arkid15r July 27, 2024 10:08
@DonnieBLT
Copy link
Collaborator

DonnieBLT commented Jul 27, 2024

Have all comments been resolved?

@Sarthak5598
Copy link
Member Author

Sarthak5598 commented Jul 28, 2024

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

Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

@arkid15r arkid15r Awaiting requested review from arkid15r

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

@Sarthak5598 @DonnieBLT @arkid15r