jfroelich / rss-reader

A simple Chrome extension for viewing RSS feeds
Other
10 stars 0 forks source link

Revise subscribe regarding observer pattern #796

Open jfroelich opened 5 years ago

jfroelich commented 5 years ago

Once subscribe does its work, it should not be concerned with notification or returning a value. Instead, it should rely on the observer pattern. Some observer somewhere else should be concerned with showing a notification upon a feed creation. Some observer somewhere else should be concerned with knowing when a subscription has completed. What it looks like is that I have followed this pattern in some places but not others.