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

2022 q4 hof #115

Merged
merged 3 commits into from
Apr 5, 2023
Merged

2022 q4 hof #115

merged 3 commits into from
Apr 5, 2023

Conversation

tomrittervg
Copy link
Collaborator

@@ -68,8 +101,7 @@ names:
date: 2022-05-16
- name: Ali Syarief
date: 2022-05-11
twitter: "@kang_ali"
url: https://twitter.com/@kang_ali
Copy link
Contributor

Choose a reason for hiding this comment

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

did Kang change the way they are mentioned?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I have an email dated Tue, Nov 22, 2022, 4:21 AM from them
 

Copy link
Contributor

Choose a reason for hiding this comment

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

got it

bug-bounty-hof/web.yml Outdated Show resolved Hide resolved
bug-bounty-hof/web.yml Show resolved Hide resolved
@@ -68,8 +101,7 @@ names:
date: 2022-05-16
- name: Ali Syarief
date: 2022-05-11
twitter: "@kang_ali"
url: https://twitter.com/@kang_ali
url: https://www.linkedin.com/in/mohammad-ali-syarief/
- name: Brian Carpenter of Geeknik Labs & Farm
Copy link
Contributor

Choose a reason for hiding this comment

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

@tomrittervg this reporter declined to be mentioned in HoF https://bugzilla.mozilla.org/show_bug.cgi?id=1796601#c10

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

He's not in Q4; this entry is for some other bug in Q2..

bug-bounty-hof/web.yml Show resolved Hide resolved
@fkiriakos07
Copy link
Contributor

Hello @tomrittervg , apologies for the delay, thank you for generating the HoF, I suggested some changes based on the reporters' requests, please check.

Copy link
Member

@dveditz dveditz left a comment

Choose a reason for hiding this comment

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

lgtm, but I don't know which concerns Frida wanted addressed

Co-authored-by: Frida Kiriakos <95376064+fkiriakos07@users.noreply.github.com>
@tomrittervg
Copy link
Collaborator Author

@fkiriakos07 Can you double check this?

@fkiriakos07
Copy link
Contributor

thanks @tomrittervg looks good to me, one question, do you know how this reporter is mentioned? https://bugzilla.mozilla.org/show_bug.cgi?id=1783267 I couldn't find any mention using his alias or name

@tomrittervg
Copy link
Collaborator Author

thanks @tomrittervg looks good to me, one question, do you know how this reporter is mentioned? https://bugzilla.mozilla.org/show_bug.cgi?id=1783267 I couldn't find any mention using his alias or name

He's in Q3 2022, double checked.

@tomrittervg tomrittervg merged commit d338aa6 into master Apr 5, 2023
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.

None yet

3 participants