Skip to content

hackdays: ai link summaries #1737

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
Draft
wants to merge 23 commits into
base: main
Choose a base branch
from
Draft

hackdays: ai link summaries #1737

wants to merge 23 commits into from

Conversation

vkraucunas
Copy link
Contributor

@vkraucunas vkraucunas commented Jun 11, 2025 •
edited
Loading

Asana Task/Github Issue: https://app.asana.com/1/137249556945/project/42792087274227/task/1210500508594526?focus=true

Description

Testing Steps

Checklist

Please tick all that apply:

  • I have tested this change locally
  • I have tested this change locally in all supported browsers
  • This change will be visible to users
  • I have added automated tests that cover this change
  • I have ensured the change is gated by config
  • This change was covered by a ship review
  • This change was covered by a tech design
  • Any dependent config has been merged

@vkraucunas vkraucunas requested a review from a team as a code owner June 11, 2025 20:07
@netlify Netlify
Copy link

netlify bot commented Jun 11, 2025 •
edited
Loading

❌ Deploy Preview for content-scope-scripts failed.

Name Link
🔨 Latest commit a5d118f
🔍 Latest deploy log https://app.netlify.com/projects/content-scope-scripts/deploys/684bb468c53aba00087ef21a

@vkraucunas vkraucunas marked this pull request as draft June 11, 2025 20:07
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 11, 2025 •
edited
Loading

Temporary Branch Update

The temporary branch has been updated with the latest changes. Below are the details:

Please use the above install command to update to the latest version.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 11, 2025 •
edited
Loading

[Beta] Generated file diff

Time updated: Thu, 12 Jun 2025 20:09:16 GMT

Android 
    - android/autofillPasswordImport.js
  • android/brokerProtection.js
  • android/contentScope.js
  • android/pages/duckplayer/dist/index.js

File has changed

Chrome-mv3 
    - chrome-mv3/inject.js

File has changed

Firefox 
    - firefox/inject.js

File has changed

Integration 
    - integration/contentScope.js
  • integration/pages/duckplayer/dist/index.js
  • integration/pages/example/dist/index.js
  • integration/pages/history/dist/index.js
  • integration/pages/new-tab/dist/index.js
  • integration/pages/onboarding/dist/index.js
  • integration/pages/release-notes/dist/index.js
  • integration/pages/special-error/dist/index.js

File has changed

Windows 
    - windows/contentScope.js
  • windows/pages/duckplayer/dist/index.js
  • windows/pages/history/dist/index.js
  • windows/pages/new-tab/dist/index.js
  • windows/pages/onboarding/dist/index.js
  • windows/pages/release-notes/dist/index.js
  • windows/pages/special-error/dist/index.js
  • windows/pages/special-error/index.html

File has changed

Apple 
    - dist/contentScope.js
  • dist/contentScopeIsolated.js
  • dist/pages/duckplayer/dist/index.js
  • dist/pages/duckplayer/index.html
  • dist/pages/history/dist/index.js
  • dist/pages/new-tab/dist/index.js
  • dist/pages/onboarding/dist/index.js
  • dist/pages/release-notes/dist/index.js
  • dist/pages/special-error/dist/index.js
  • dist/pages/special-error/index.html

File has changed

@vkraucunas vkraucunas self-assigned this Jun 11, 2025
jonathanKingston
jonathanKingston reviewed Jun 12, 2025
View reviewed changes
mgurgel and others added 15 commits June 12, 2025 09:46
@mgurgel
[Hackday] Hover SummarizationRunning on integration (#1741)
e460141 
* Running on integration

* Running on integration

* Removed extra console.log
@vkraucunas
chore: Add integration view for faster UI work
357cdb2
@vkraucunas
chore: Add new msgs
5d2881b
@vkraucunas
chore: start building card
ad10b19
@vkraucunas
fix: support for relative path links and no protocols
4f13242
@vkraucunas
chore: Add timeToRead info, turn on summarization msg
5a90083
@vkraucunas
chore: text update
7795fd8
@vkraucunas
feat: Add tiered loading support, start arranging UI blocks
e47d23f
@vkraucunas
styles!
200d919
@vkraucunas
fix: use real data
ef832df
@vkraucunas
fix: border under list
aa9d5d1
@vkraucunas
feat: Add disclaimer text
f5186bd
@vkraucunas
small style improvement.
88f897f
@greyivy
Put it in an iframe (so styles wouldn't be overwritten), glued it to …
a367dd9 
…the element using Popper, and tweaked a couple styles
@greyivy
Inject deps to prevent cross origin errors
a5d118f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonathanKingston jonathanKingston jonathanKingston left review comments

@kzar kzar Awaiting requested review from kzar kzar will be requested when the pull request is marked ready for review kzar is a code owner

@sammacbeth sammacbeth Awaiting requested review from sammacbeth sammacbeth will be requested when the pull request is marked ready for review sammacbeth is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@vkraucunas vkraucunas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vkraucunas @jonathanKingston @mgurgel @greyivy