-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add support for TCFv2 and modernize the web extension #10
base: master
Are you sure you want to change the base?
Conversation
charles-tan
commented
Mar 30, 2022
- Call TCFv2 APIs
- Refactor of backend
- Modernize the UI and expose more info returned in the TC string
clean up files
fix TCFv2 vendor list request
Update script to use tcfv2 API's
[#5] add state for gdprApplies
…less if cmp locator is found
…ie_code_flow Fixes #4 by cleaning up code and adding listener in uCookie.js regardless if cmp locator is found
Related to Issue #6 to properly load the vendorsList
[#15] set up boilerplate for modules & hot reloading
update README and cmp list
fix special features text
add css fixes for firefox
make table header sticky
update installation instructions
clean up codebase
gdpr applies fix
Hello,
The rest is minor and I can fix it later. Thanks! |
Hi, it's been quite some time since I looked at this and I've lost some context. I'll try to revisit this and look into cleaning up some dependencies. We finished this project in a bit of a hurry and I'm sure there are unused dependencies that we might have explored using and never cleaned up. I was able to rebuild from a fresh setup. I'm using I recall that we used yarn because we found it easier for our dependency management, we were running into some issues with plain npm. Beyond that, I can't remember unfortunately. Is yarn something you're trying to avoid? |
Webpack is not strictly necessary, but we found it to be useful for development. The webserver in I apologize for the messy commit history... I tracked down the old PR where this change was implemented. There is some good context there behind our rationale for using yarn + webpack. |
Hello, I have the following versions:
I'll come back to this a bit later, but I may remove webpack, especially since most dependencies are gone. Don't worry about the commit history, I'll probably squish everything into a single commit when merging |
So, some updates as I've spent some time trying to integrate this:
Honestly, I don't think I'll spend more time working on this as this project is no longer important to me. |