Skip to content
This repository has been archived by the owner on Jun 7, 2022. It is now read-only.

Stop showing actioned inbox items #8394

Merged
merged 2 commits into from Mar 1, 2022
Merged

Conversation

rjchow
Copy link
Contributor

@rjchow rjchow commented Mar 1, 2022

Fixes #8393
Revert of #7983

Detailed test instructions:

  1. Navigate to WooCommerce -> Home
  2. Click on a note action (such as "Learn more") to make it actioned
  3. Reload the page.
  4. Make sure the note you just clicked is no longer present.

@rjchow rjchow force-pushed the fix/hide-actioned-notifications branch from fd1b21f to ca275a5 Compare March 1, 2022 12:06
@rjchow rjchow marked this pull request as ready for review March 1, 2022 12:06
@rjchow rjchow added focus: inbox Issues related to inbox notifications status: needs review labels Mar 1, 2022
@ilyasfoo ilyasfoo self-requested a review March 1, 2022 13:07
Copy link
Contributor

@ilyasfoo ilyasfoo left a comment

Choose a reason for hiding this comment

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

Tested well, LGTM!

@ilyasfoo
Copy link
Contributor

ilyasfoo commented Mar 1, 2022

Merging it now to include in WCA 3.3 beta 1.

@ilyasfoo ilyasfoo merged commit ab7be49 into main Mar 1, 2022
@ilyasfoo ilyasfoo deleted the fix/hide-actioned-notifications branch March 1, 2022 13:08
ObliviousHarmony pushed a commit to woocommerce/woocommerce that referenced this pull request Mar 10, 2022
* Stop showing actioned inbox items

- revert of woocommerce/woocommerce-admin#7983

* Trigger rerun of CI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
focus: inbox Issues related to inbox notifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inbox: reintroduce the auto dismiss behavior
2 participants