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

Save Summary Type & Target Language #46

Merged
merged 2 commits into from
Nov 5, 2023
Merged

Conversation

BrunoBernardino
Copy link
Contributor

@BrunoBernardino BrunoBernardino commented Nov 3, 2023

This implements saving of the summary type and target language choices so they can be used for the next summarization (including via shortcut).

It effectively closes the request at https://kagifeedback.org/d/2321-chrome-extension-summarizer-extension-always-uses-summary-instead-of-key-moments

It also updates the expected node and npm versions for development (current LTS) and the nodemon development dependency, to resolve a security vulnerability without having to use package.json:overrides.


kagi_chrome_0.4.2.zip

kagi_firefox_0.4.2.zip

This implements saving of the summary type and target language choices so they can be used for the next summarization (including via shortcut).

It effectively closes the request at https://kagifeedback.org/d/2321-chrome-extension-summarizer-extension-always-uses-summary-instead-of-key-moments

It also updates the expected `node` and `npm` versions for development (current LTS) and the `nodemon` development dependency, to resolve a security vulnerability without having to use `package.json:overrides`.
@BrunoBernardino BrunoBernardino self-assigned this Nov 3, 2023
@vprelovac vprelovac merged commit ee17ff1 into main Nov 5, 2023
1 check passed
@BrunoBernardino BrunoBernardino deleted the feature/save-summary-type branch November 5, 2023 06:54
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.

2 participants