Skip to content
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

Add new page for Foundations Pluggable Triggers #1

Merged
merged 4 commits into from
Jun 28, 2024

Conversation

mellisoncertinia
Copy link
Collaborator

The Winter '23 release of the Certinia Foundations package added a 'Pluggable Trigger' framework. We want to provide additional documentation to support its use.

@rcorfieldffdc
Copy link
Contributor

rcorfieldffdc commented Jun 13, 2024 via email

@mellisoncertinia
Copy link
Collaborator Author

Sounds good. Are these versioned, so a customer sees which version the text applies to?

On Thu, 13 Jun 2024 at 14:51, Martyn Ellison @.> wrote: Non-Certinia Sender------------------------------ @.* commented on this pull request. ------------------------------ In pluggable-triggers.md <#1 (comment)> : > + when BEFORE_DELETE, AFTER_DELETE { + records = Trigger.old; + } + when else { + records = Trigger.new; + } + } + + + try { + fferpcore.PluggableTrigger target = + new fferpcore.PluggableTrigger.Constructor() + .construct(records); + + + switch on Trigger.operationType { That's a good point. I think with the new runTriggerHandler() method we can greatly simplify the Apex trigger code snippet provided here. Are you happy if I update this on/after Summer '24 ERD? — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKIBRMF7JGKNS7UUE3SBXLLZHGPWZAVCNFSM6AAAAABJED3ADSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMJVHA3TEMBTHE . You are receiving this because your review was requested.Message ID: @.***>

The developers site isn't versioned so the documentation targets the most recent FDN package. I've add a sentence to make this clear. I don't expect the pluggable trigger framework to change much so hopefully we won't need versioning. From Summer '24 the products using the framework will publish details of their pluggable triggers in the technical pack (which is versioned). We'll be able to remove the 'Existing Pluggable Triggers' table and direct everyone to the technical pack.

@dbtavernerffdc dbtavernerffdc merged commit a68045d into main Jun 28, 2024
@dbtavernerffdc dbtavernerffdc deleted the feature/pluggable-triggers branch June 28, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants