-
Notifications
You must be signed in to change notification settings - Fork 238
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
Updated API versions from v58.0 to v59.0 #1397
Updated API versions from v58.0 to v59.0 #1397
Conversation
Thanks for the contribution! Before we can merge this, we need @groundwired to sign the Salesforce Inc. Contributor License Agreement. |
This is great but it is actually time to bring everything up to API V59.0, would you be willing to change this to V59.0? If you can cleanup the extra file and sign the CLA that should be everything we need to accept this! Really appreciate you bringing a problem and a solution. |
@aheber Should be all set! |
This looks great, thank you for expanding it! You even caught all the non-xml versions that most people miss. I'll have to figure out what we need to do to get the automated tests to run so we can accept this. I think we might need to recreate the branch inside the team-scope but I'll get back to you on that. |
That seems wise! FWIW, I did run all Apex tests in the package in my scratch org. Everything passes. |
@groundwired I think you should have received an invite to join the project. That should allow you to close this PR and your branch under your name. Push these changes to a branch directly on the project an open a new PR. That will allow the automation to run and validate your changes and get the needed passes so we can merge. Sorry for the trouble but hopefully it isn't a lot of work. |
Critical Changes
RollupDmlGuard API version updated to v58.0
Changes
No code changes - just version updates in two files that were missed
Issues Closed
#1396