Skip to content

Refactor: PB store and provider#1092

Merged
mohdsayed merged 6 commits into
developfrom
ref/prebid-separation
Jun 9, 2025
Merged

Refactor: PB store and provider#1092
mohdsayed merged 6 commits into
developfrom
ref/prebid-separation

Conversation

@amovar18

@amovar18 amovar18 commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator

Description

This PR separates PB store and provider, providing more readability in the code.

Relevant Technical Choices

Testing Instructions

This is a code refactoring change, hence all the working of the extension should remain the same.

Additional Information:

Screenshot/Screencast


Checklist

  • I have thoroughly tested this code to the best of my abilities.
  • I have reviewed the code myself before requesting a review.
    - [ ] This code is covered by unit tests to verify that it works as intended. NA
  • The QA of this PR is done by a member of the QA team (to be checked by QA).

@amovar18 amovar18 self-assigned this Jun 5, 2025
@amovar18 amovar18 added this to the v1.0 milestone Jun 5, 2025
@amovar18 amovar18 moved this to In QA in PS Analysis Tool Jun 5, 2025
@amovar18 amovar18 marked this pull request as ready for review June 6, 2025 07:30
@amovar18 amovar18 requested a review from mohdsayed June 6, 2025 07:30
@amovar18 amovar18 moved this from In QA to Under Review in PS Analysis Tool Jun 6, 2025

@mohdsayed mohdsayed left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mohdsayed mohdsayed merged commit 3127066 into develop Jun 9, 2025
6 checks passed
@github-project-automation github-project-automation Bot moved this from Under Review to Done in PS Analysis Tool Jun 9, 2025
@mohdsayed mohdsayed deleted the ref/prebid-separation branch June 9, 2025 07:31
@mohdsayed mohdsayed mentioned this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants