-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
Implement relatedLogEntries refresh button + spinner #703
Conversation
Hi @andrii-solokh - I definitely like the idea of having a refresh button on the component, but I think using CDC is going to cause too many other problems:
So I think the CDC-related changes needed to be removed, it should just be a simple refresh button (that a user would manually click to refresh). Let me know if that makes sense/if you have any questions. |
Hi, @jongpie. Thank you for response, I can't find docs telling 5 objects are the limit, only number of concurrent subscribes. But, okay i will remove CDC part. |
Hi @andrii-solokh - apologies, I meant to include a source for the 5 object limit. It's documented here on the page 'Default Change Event Allocations' You can also test out the limit in a scratch org/sandbox - if you try to add more than 5 objects to CDC, you'll get this error: |
08ae2e7
to
ff2b7b2
Compare
ff2b7b2
to
731e795
Compare
@jongpie, thank you for the explanation, it looks like I should be more careful with using CDC and events in general. Removed CDC part. |
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.
@andrii-solokh thanks so much for working on this, and for removing the CDC-related changes. I think what you built using CDC was very cool (and a great use case for CDC), but since Salesforce's platform limitations for it are so low, it's hard to rely on it sometimes.
Your changes look great - I've added a few extra commits to create the new package version, and I'll be releasing this today as v4.13.12
.
@andrii-solokh this now has been officially released as Thanks again for taking the time to work on this, I really appreciate the help! |
@jongpie it was tiny one, I hope to be more helpful in future 👍🏼 |
Motivation
Working on new automation on a client project, I was generating logs in apex and attaching them to records. Each time I need to refresh the whole page, it was a bit annoying, so I thought it would be better to have a button for refresh or to auto refresh when new log created for currently open record.
Solution
Changes
Demo
Changing account name generates log on record.
demo.mov