Aleksej Vukomanovic comments

Results 128 comments of


                                            Aleksej Vukomanovic

Raise minimum required version of PHP to 7.4

Just to note that #8094 was merged very recently, it seems that package can be upgraded also as part of this issue

Raise minimum required version of PHP to 7.4

@aaemnnosttv > I'm not sure this requires any change for us, since the test referenced here is using a non-proxy-connected scenario. When a site is connected to the proxy, we...

Enhancement/9162 Top Device Driving Purchases KMW

One note here - before this PR is merged, this tile should be included in the list of tailored metrics to the designated purpose answers, follow the table in AC...

Update `subtitle` in the widget area API to accept component or text as parameter and use it for KMW area

Thanks @eugene-manuilov AC updated

Add new resolver to the conversion reporting datastore partial to retrieve detected and lost events from inline module data

@kelvinballoo Thanks for you observations. Sorry about that, I update QAB now, we changed in the last CR round, what you see is correct output, `hasNewConversionReportingEvents` and `hasLostConversionReportingEvents` return `true`,...

Add new badge to the ACR widget tiles and groups in key metrics selection panel

Thanks @kelvinballoo > This might be out of the scope for this ticket but highlighting in case we need to create another task. > I noticed one difference when I...

Create `ChipTabGroup` component

@eugene-manuilov AC updated. Thanks

Create `ChipTabGroup` component

@eugene-manuilov > let's rename active to be isActive and instead of using the group property, let's use slug and label props. Updated, thanks. > can you break down the estimate...

Create `ChipTabGroup` component

Hi @kelvinballoo thanks for your observations, > ITEM 2: Should we be implementing the dots at the top of groups? Image attached for reference. This will be implemented in the...

Expand the logic of the `ACRNotificationCTAWidget` component for existing KMW users that already have ACR tiles set up

@10upsimon Yes the explained steps are correct and expected behaviour. After new events are detected following initial detection, this banner is shown with same CTA regardless of how KMW was...