New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #3080: Activity Feed: Blank feed on my-data page for updating user's role or team #3131
Conversation
… updating user's role or team
for (var thread : threads) { | ||
feedDao.create(thread); | ||
// Skip creating threads for user or team related changes for now | ||
if (!Entity.USER.equals(changeEvent.getEntityType()) && !Entity.TEAM.equals(changeEvent.getEntityType())) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we maintain a list of entities in Entity.java that we can grab with shouldDisplayEntityChangeOnFeed
or something similar?
We need to avoid user, team, role, bot, policy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. I will work on updating the patch.
[catalog] Kudos, SonarCloud Quality Gate passed! |
// | ||
// List of entities whose changes should not be published to the Activity Feed | ||
// | ||
public static final List<String> ACTIVITY_FEED_EXCLUDED_ENTITIES = List.of(USER, TEAM, ROLE, POLICY, BOTS); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor: this could be a Set/HashSet
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
Test failure is not related
|
Describe your changes :
See #3080
Type of change :
Checklist:
Reviewers