fixed a bug #110

gabrielflorit · Nov 22, 2014 · cb58189 · cb58189
1 parent be6e7ef
commit cb58189
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/js/Updates.jsx b/src/js/Updates.jsx
@@ -83,11 +83,21 @@ var Updates = React.createClass({
 	},
 
 	// Handle clicking the close button.
-	onButtonClick: function(dismissedUpdateNumbers) {
+	onButtonClick: function(newUpdateNumbers) {
 
 		// Save dismissed update numbers in local storage, so we don't
 		// display them again.
-		localStorage.setItem(Updates.DISMISSED_UPDATE_NUMBERS, dismissedUpdateNumbers);
+		var dismissedUpdateNumbers = [];
+		var localNumbers = localStorage.getItem(Updates.DISMISSED_UPDATE_NUMBERS);
+		if (localNumbers) {
+			dismissedUpdateNumbers = localNumbers.split(',').map(function(v) {
+				return +v;
+			});
+		}
+
+		var allDismissedNumbers = _.uniq(dismissedUpdateNumbers.concat(newUpdateNumbers));
+
+		localStorage.setItem(Updates.DISMISSED_UPDATE_NUMBERS, allDismissedNumbers);
 
 		// Change this component's state to closed. This will trigger a
 		// re-render.