search

woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
277 stars 135 forks source link

[Notifications] Replace mutable map with Thread safe map for storing all active notifications #12654

Open AnirudhBhat opened 2 months ago

AnirudhBhat commented 2 months ago

While fixing ConcurrencyModificationException https://github.com/woocommerce/woocommerce-android/pull/12646, we came to conclusion that replacing mutableMap with Thread safe map would benefit in the long run. This ticket is to do that.

dangermattic commented 2 months ago

🚫 Please add a type label (e.g. type: enhancement) and a feature label (e.g. feature: stats) to this issue.

wpmobilebot commented 1 month ago

Version 20.6 has now entered code-freeze, so the milestone of this PR has been updated to 20.7.

wpmobilebot commented 1 month ago

Version 20.7 has now entered code-freeze, so the milestone of this PR has been updated to 20.8.

wpmobilebot commented 1 month ago

Version 20.8 has now entered code-freeze, so the milestone of this PR has been updated to 20.9.

wpmobilebot commented 4 weeks ago

Version 20.9 has now entered code-freeze, so the milestone of this PR has been updated to 21.0.

wpmobilebot commented 3 weeks ago

Version 21.0 has now entered code-freeze, so the milestone of this PR has been updated to 21.1.

wpmobilebot commented 2 weeks ago

Version 21.1 has now entered code-freeze, so the milestone of this PR has been updated to 21.2.

wpmobilebot commented 12 hours ago

Version 21.2 has now entered code-freeze, so the milestone of this PR has been updated to 21.3.