-
-
Notifications
You must be signed in to change notification settings - Fork 189
feat(chrome_extension): write dom changes to database (using sockets) during openadapt.record #364
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
Conversation
Merge main into test_extension
@0dm @KrishPatel13 can you please remind me what is the status on this? |
from openadapt import config, sockets | ||
|
||
SOCKETS = True | ||
DBG_DATABASE = False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please clarify what this is for?
|
||
from openadapt import config, sockets | ||
|
||
SOCKETS = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this ever be False?
@KrishPatel13 I believe this should be closed in favor of #744, can you please confirm? |
Yes!, closing as a new PR is launched for this here: #744 |
What kind of change does this PR introduce?
This PR implements the chrome extension to write the DOM changes (documents' outer HTML of Head and Body and the url) in database.
Summary
Motivation: Related Issue: #51
Checklist
How can your code be run and tested?
Other information