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

Added CVE and EPSS to Chrome js #33

Closed
wants to merge 1 commit into from
Closed

Conversation

chooko
Copy link

@chooko chooko commented Jan 25, 2024

Hello Mitch,

We emailed briefly this morning about adding CVEs and EPSS to the extension. I decided to take a shot at it myself once I saw how the code was structured. Can you review?

(FWIW: I tried to add to the Firefox contexts as well, but when I realized they require a png it was a bit more than I wanted to bite off for this session.)

Thanks!!

@chooko
Copy link
Author

chooko commented Feb 6, 2024

@mitchmoser Looks like my checks have passed. Any luck that you can review this and push to next app update? 😊

@mitchmoser
Copy link
Owner

mitchmoser commented Feb 6, 2024

Hi Brayden, I believe we had discussed tracking this request as an Issue rather than opening a PR.
The rationale is similar to Issue #15 with a request for a new search category that would consist of a single OSINT source.
I will close this PR for now and open an Issue to track and compile new sources for this category until there are enough sources that a new category should be implemented into the extension.
I am certainly open to answering any additional questions or taking feedback surrounding this.

@mitchmoser mitchmoser closed this Feb 6, 2024
@mitchmoser mitchmoser added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants