[$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

[yoution]

1. seems still not work
   
   
2. for ba id
   
   

[maxceem]

1. workPeriods.isFirstWeek and workPeriods.isLastWeek are not allowed inside the fields param, because we don't have such fields. We can only use them as filters.

2. hmm, I'll check this out

[maxceem]

@yoution regrading the second issue with BA. It happens because &billingAccountId=0 is included twice inside the request which is converted to array in the API side. &billingAccountId=0 should be included only once in URL.

[maxceem]

Challenge https://www.topcoder.com/challenges/e873575e-b876-4f19-a499-17262a0c76cc has been created for this ticket.

This is an automated message for maxceem via Topcoder X

[maxceem]

Challenge https://www.topcoder.com/challenges/e873575e-b876-4f19-a499-17262a0c76cc has been assigned to yoution.

This is an automated message for maxceem via Topcoder X

[maxceem]

Payment task has been updated: https://www.topcoder.com/challenges/e873575e-b876-4f19-a499-17262a0c76cc
Payments Complete
Winner: yoution
Copilot: maxceem
Challenge e873575e-b876-4f19-a499-17262a0c76cc has been paid and closed.

This is an automated message for maxceem via Topcoder X

[sandhiyakavi]

Verified in Dev Env.

1. If all Alert options selected, only resource that satisfies all options is displayed
   

2. If 'Last Booking Week' selected, only resources with last booking week is displayed,
   

3. If 'No BA assigned' selected, only resources for which BA not assigned are displayed,
   

4. No resource booking found displayed if no resource booking satisfies the options.
   

[lakshmiathreya]

Filter by Alerts Verified in Prod.