Skip to content

Misc QA Fixes#1043

Merged
mohdsayed merged 11 commits into
release/v0.14.1from
fix/classifier-minor-style
Apr 14, 2025
Merged

Misc QA Fixes#1043
mohdsayed merged 11 commits into
release/v0.14.1from
fix/classifier-minor-style

Conversation

@amovar18

@amovar18 amovar18 commented Apr 10, 2025

Copy link
Copy Markdown
Collaborator

Description

  • Regression: In multi-seller auction, the top level auction duplicates the first component auction data.
  • Minor styling issue when resizing DevTools, the icon slides down for No ad Unit in auction tab.
  • Minor styling issue text colour changes to black for CDP on/off message for cancel button
  • Enhancement: Topics > Classifier - If an incorrect domain is submitted, the table data should be rest if any previous data exists

Relevant Technical Choices

Testing Instructions

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.
  • The QA of this PR is done by a member of the QA team (to be checked by QA).

CharliePops and others added 11 commits April 2, 2025 11:09
* make json view collapsible and replace third party library

* improve colors and fix dark mode

* rename json theme file

* fix import statement

* fix puppeter and worker.js

* expand only root object in json view

* improve dark light color matching

* further improve styles to match default devtools styling

* Reduce the size of three dots in collapsed state

* center loading text and center collapse/expand icons

---------

Co-authored-by: sayedtaqui <sayedwp@gmail.com>
* set theme mode dynamically

* fix dark/light mode in jsonview

* remove console.log

* fix broken test

* improve theme mode set logic
@mohdsayed mohdsayed changed the title Fix: Style fixes Misc QA Fixes Apr 14, 2025
@amovar18 amovar18 self-assigned this Apr 14, 2025
@amovar18 amovar18 added the bug Something isn't working label Apr 14, 2025
@amovar18 amovar18 moved this to In QA in PS Analysis Tool Apr 14, 2025
@amovar18 amovar18 added this to the v0.14.1 milestone Apr 14, 2025
@amovar18 amovar18 marked this pull request as ready for review April 14, 2025 08:16
@amovar18 amovar18 requested a review from mohdsayed April 14, 2025 08:22
@mohdsayed mohdsayed merged commit c6120ce into release/v0.14.1 Apr 14, 2025
@github-project-automation github-project-automation Bot moved this from In QA to Done in PS Analysis Tool Apr 14, 2025
@mohdsayed mohdsayed deleted the fix/classifier-minor-style branch April 14, 2025 08:24
@mohdsayed mohdsayed mentioned this pull request Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants