Update monitor annotations job to use contentType=dataset filtering

[anngvu]

Context and user story:

In September 2022, the first version of annotations reminder was implemented via #25.
In March 2023, we switched over to schematic's preferred way of using contentType=dataset in #50.

The logic of monitor annotations currently aligns more with ProjectLive, which does not use the contentType=dataset to do a pre-selection of data assets first; selection of data to pay attention to is primarily based on resourceType. But ProjectLive will soon go away anyway, to be replaced by DataFlow, and the views will look different and indeed be more focused on contentType=dataset folders. So it is likely better if reminders be consistent with DataFlow (and DCA) by using the contentType=dataset selection approach.

Other reasons to revise approach:

• Simpler and easier to take advantage of labels added on folders that weren't there before, and reduce the current inferencing/pattern matching needed.

Raised by @cconrad8, open for comments by @allaway and @jaybee84.

[allaway]

I agree with this. Thanks!

…

On Mon, Mar 4, 2024 at 6:37 PM Anh Nguyet Vu ***@***.***> wrote: Context and user story: In September 2022, the first version of annotations reminder was implemented via #25 <#25>. In March 2023, we switched over to schematic's preferred way of using contentType=dataset in #50 <#50>. The logic of monitor annotations currently aligns more with ProjectLive, which does not use the contentType=dataset to do a pre-selection of data assets first; selection of data to pay attention to is primarily based on resourceType. But ProjectLive will soon go away anyway, to be replaced by DataFlow, and the views will look different and indeed be more focused on contentType=dataset folders. *So to consistent, it is likely better if reminders be consistent with DataFlow (and DCA) by using the contentType=dataset selection approach.* Raised by @cconrad8 <https://github.com/cconrad8>, open for comments by @allaway <https://github.com/allaway> and @jaybee84 <https://github.com/jaybee84>. — Reply to this email directly, view it on GitHub <#81>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AE3WNSBGWJXRY7BLFZMV3LTYWUHTNAVCNFSM6AAAAABEGFT4FGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DQMBUHE3DKNA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[cconrad8]

I also agree! Thanks for writing it out