[$50] Filter by Alerts

[maxceem]

Support filtering by Alerts column.

On the left sidebar add a new section called Alerts with the next 3 checkboxes to filter by:

• No BA Assigned
• OnBoarding Week
• Last Booking Week

It should look like this:

API

• API is updated to support these filters, see Add new filter to the GET Resource Bookings endpoint topcoder-platform/taas-apis#409 for details.

[maxceem]

@yoution the API for this task is implemented now, so feel free to pick this up.

[yoution]

@maxceem please assign to me

[yoution]

@maxceem I don't know how to pass the parameters to the BE，
for alert filter, we can only select one option at the same time? or we can select all like payment status ?
for no db assign, if checked, we pass billingAccountId:0, if unchecked? what to pass?
for onboarding week, if checked, we pass isFirstWeek=true, if unchecked, we pass isFirstWeek=false? and last booking week also do like so?

[maxceem]

@yoution good questions.

for no db assign, if checked, we pass billingAccountId:0, if unchecked? what to pass?

if unchecked, then don't pass filter by billingAccountId at all

for onboarding week, if checked, we pass isFirstWeek=true, if unchecked, we pass isFirstWeek=false? and last booking week also do like so?

if uncheck, better don't pass anything, but passing false is also fine.

for alert filter, we can only select one option at the same time? or we can select all like payment status ?

Hmm, I have to clarify this moment. I would be back to you shortly. For now, I guess you can allow to select them all like for payments.

[yoution]

because I found if I pass isFirstWeek and isLastWeek together, the BE will show error, so I asked to you.
on the other hand, this payment seems a little low, the url must update, when I check the checkbox, how about 50$?

[maxceem]

because I found if I pass isFirstWeek and isLastWeek together,

yeah, we would fix it in API, please ignore this error for now, it would go away soon.

on the other hand, this payment seems a little low, the url must update, when I check the checkbox, how about 50$?

ok, updated to $50

[maxceem]

because I found if I pass isFirstWeek and isLastWeek together,

API should allow passing both together now