-
Notifications
You must be signed in to change notification settings - Fork 49
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
FED-1436 feat (logging) adds logging for health monitoring to CMP to verify init and init success flow #149
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* (docs) testing pr template * (docs)
* (refactor) partial migration * (migrate) upgrade to latest cmp and move to openmail repo * (refactor) fix webpack dev server for s1 * (refactor) add cross-var and update deploy:s1 npm scripts * (feat) deploy:s1 * (refactor) move to configure dir * (refactor) reorganize files * (refactor) deploy s1 and deploy original separately to s3 bucket * (refactor) reogranize loader files and update deploy * (feat) added .node_version * (refactor) remove codesmell from package json and babel config
* Add gdprAppliesGlobally flag as independent config property * Truncate metadata string to match spec * Update a couple dependencies
* upgrade (all) update to 89504d7 * upgrade cmp * update style vars * upstream merge and handle combining with customization from modal branch * fix tests * bump * bump * fix pubVendorConsent to check against known vendors * adds footer / removes import tests because import isnt implemented correctly * remove FOUC
…ns per vendor (#16) * feat (ui: features) add ui for features names and descriptions per vendor * remove console logs
…st in first UI layer (#17)
…or (#18) * FED-1221 refactor (init: events) only emit consent-change when no error * add more tests * add option to auto-upgrade consent with footer * docs and tests * fix onConsent event * fix auto quotes * fix bracket spacing
* FED-1220 feat (cmp: modal) add modal option and update styling * docs * auto consent on esc * update changelog * bump * fix ie11 * remove logs * remove logs * turn off logging
… instead of Footer (#20)
…f callback and consent-change trigger (#21) * NS-1963 fix (cmp: cookie) coerce cookie to boolean and update order of callback and consent-change trigger * add tests * readme * update autoformatting * fix styling * comment out language option
… cookie Domain (#22) * feat (cookie) remove domain inference and expose config option to set cookie Domain * bump * update tests to use co.uk
…ry domain (#23) * refactor (cookie) allow wildcard cookie domain on naked primary domain * remove unused
feat(system1-cmp): Add call to ReactNativeWebView
…info-acs reference implementation (#27) * refactor (reference: info-acs) refactor build and add themes for the info-acs reference implementation * changelog * readme * fix loaders * add depth to card * fix tests
…sent, and submit (#28)
fix(info-acs): Fix ReactNative detection
… on allowedVendorIds (#30) * refactor (store: features, vendors) filter vendors from display based on allowedVendorIds * fix lint * fix eslint prettier conflict * fix eslint - dont allow interactive custom vendors
…verify init and init success flow
fml - i dunno why it defaults to a PR to primary branch - sorry |
@potench you could remove this repo as a remote and then you won't accidentally push here anymore. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
background
test plan