You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the context of #3746, we want to make common not depend on datastore. One place it does heavily is in the DatasetInfo class. Let's make sure this service can work if datastore is not enabled, and simply return less info.
In the future, we should either set up some kind of plugin or subscriber system so that any module can add info to the core dataset info object. OR (even better) we should maybe find a way to stop using this class altogether. It is kind of an anti-pattern.
The text was updated successfully, but these errors were encountered:
In the context of #3746, we want to make
common
not depend ondatastore
. One place it does heavily is in theDatasetInfo
class. Let's make sure this service can work if datastore is not enabled, and simply return less info.In the future, we should either set up some kind of plugin or subscriber system so that any module can add info to the core dataset info object. OR (even better) we should maybe find a way to stop using this class altogether. It is kind of an anti-pattern.
The text was updated successfully, but these errors were encountered: