Hide System Field Updates in Record Page Timelines

[Bonapara]

Current Behavior

Some system field updates are displayed in record page timelines. However, as they lack a FieldDisplay, no value appears near the field title.

Desired Behavior

We shouldn't display system fields in the timeline.

[Faisal-imtiyaz123]

@Bonapara I would like to work here.

[Bonapara]

Sure @Faisal-imtiyaz123, thanks for contributing!

[charlesBochet]

@Bonapara @FelixMalfait should we rather add a isAuditable on field level?

I don't like the isSystem one :p
We have a settings column on Fields, we plan to add things in it such as: isSearchable, we could add isAuditable there too

[FelixMalfait]

@charlesBochet At the field-level it's a bit more subtle than isAuditable because we still want these fields to be audit-logged but we don't want the timelineActivity to display include them, so I'd advocate for a different name (isDisplayedInTimelineActivity or something like that).
But I don't see a case where a field would be displayed in the page, and wouldn't be displayed in the timeline.
So in my opinion, the field we should use here is the same we will introduce to replace most use-cases for isSystem (isDisplayed or something like that).
Bottom line is I think we should use isSystem

[charlesBochet]

Okay I understand the reasoning! Let's go for isSystem as long as we don't have a better naming

[FelixMalfait]

Changing to GoodFirstIssue as it shouldn't be too difficult.
We should filter out fields where isSystem = true in the diff.
Think about the edge case where just 1 field was changed (e.g. position was changed by dragging the item on the list view), then we don't want to create any timelineActivity.

Do it on the backend rather than on the frontend. I would probably do it in EntityEventsToDbListener