Skip to content

[feat] V2- Support Incoming MFE #50

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
Open
FentPams wants to merge 18 commits into
base: v2
Choose a base branch
from
Open

[feat] V2- Support Incoming MFE #50

FentPams wants to merge 18 commits into from

Conversation

@FentPams
Copy link
Contributor

@FentPams FentPams commented Mar 26, 2025

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@FentPams FentPams changed the base branch from main to v2 March 26, 2025 00:29
@FentPams FentPams changed the title [feat] V2- Support Incoming MFE (WIP) [feat] V2- Support Incoming MFE Mar 26, 2025
@FentPams FentPams added the WIP still work in progress label Mar 26, 2025
@codecov
Copy link

codecov bot commented Apr 1, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 20.90909% with 87 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/index.js 20.90% 80 Missing and 7 partials ⚠️

📢 Thoughts on this report? Let us know!

ramboz and others added 15 commits May 28, 2025 23:27
@ramboz
feat: pass prodHost value to the experimentation rail
950b841
@FentPams
add new optimizingTarget attribute in config
6feefdb
@FentPams
move event listener to loadeager
004c54c
@FentPams
add metadata label into config for new rail to display
f9fd60a
@FentPams
feat:implement consent based experiment
37ba23f
@FentPams
fix lint
839aed3
@FentPams
update readme installation part for new ui
7049c08
@FentPams
Update consent management to only store explicit consent
acc7312 
- Store consent in localStorage only when explicitly granted
- Remove consent from storage when denied or revoked
- Add debug logging for consent grant and revocation
@FentPams @ramboz
Update README.md
0172048 
Co-authored-by: Julien Ramboz <ramboz@users.noreply.github.com>
@FentPams
address comments
4d1335f
@FentPams
update changes from consent management
5483927
@FentPams
update readme
c688a15
@FentPams
update readme
0475de6
@FentPams
update readme
127e270
@FentPams
address lint error and update test about consent management
89b43f5
@FentPams FentPams marked this pull request as ready for review December 5, 2025 22:21
@FentPams FentPams requested a review from ramboz December 5, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ramboz ramboz Awaiting requested review from ramboz

Assignees

No one assigned

Labels

WIP still work in progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@FentPams @ramboz