Merge master into feature/guided-flow#604
Open
tcdahlberg wants to merge 103 commits intofeature/guided-flowfrom
Open
Merge master into feature/guided-flow#604tcdahlberg wants to merge 103 commits intofeature/guided-flowfrom
tcdahlberg wants to merge 103 commits intofeature/guided-flowfrom
Conversation
removed field check in test
…ment-error 533 fix required appointment error
Fix small typo
…ate/ctry picklist org enabled
…nts-App into person_account_matching2
…ndering-in-SummitEventsRegisterAppointments 588 fix bullet points rendering in SummitEventsRegisterAppointments
Appointments auto add failing
New version of SummitEventsShared.getDependentSelectOptions()
…ent, not capacity
…and lead conversion
Description to use: Adding ECCN information per Salesforce OSPO request
Added documentation outlining the security requirements for Connected Apps and External Client Apps in relation to the Summit Events App.
Update CODEOWNERS
arvindbhamidipati
temporarily deployed
to
github-pages
NEW FEATURE: Person account matching
There was a problem hiding this comment.
Pull request overview
This pull request merges the master branch into the feature/guided-flow branch. It's an automated merge that includes significant updates to the Summit Events application, including API version updates, new Experience Bundle configurations, person account matching capabilities, code refactoring, and deprecation of legacy functionality.
Key changes:
- API version upgrade from 61.0 to 64.0 across all Apex classes and pages
- Introduction of Experience Bundle (Salesforce Community) configuration for "Summit Events"
- New person account matching functionality with custom metadata and fields
- Code refactoring to consolidate query logic into a new
SummitEventsReadSharedclass - JavaScript modernization (var to let) and bug fixes
- Deprecation of parking pass and host assignment features
Reviewed changes
Copilot reviewed 194 out of 199 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| unpackaged/pre/roles/* | New role definition for Summit Events Community Owner |
| unpackaged/config/site/* | Site configuration changes, migration to ChatterNetwork type |
| unpackaged/config/experiences/Summit_Events1/* | New Experience Bundle with views, routes, themes, and configurations for community portal |
| sfdx-project.json | API version update to 64.0 |
| robot/SummitEventsApp/tests/* | New and updated Robot Framework test files |
| orgs/*.json | New and updated org configuration files with features and settings |
| force-app/test/default/classes/* | API version updates and test improvements across all test classes |
| force-app/main/default/classes/SummitEventsReadShared.cls | New shared class to consolidate SOQL queries |
| force-app/main/default/classes/SummitEventsRegisterController.cls | Refactored to use new read shared class, bug fixes |
| force-app/main/default/classes/SummitEventsShared.cls | Refactored logic, new person account helper, dependent picklist improvements |
| force-app/main/default/staticresources/SummitEventsAssets/js/*.js | JavaScript modernization (var to let), null check fixes |
| force-app/main/default/objects/Summit_Events__c/fields/* | New fields for person account and guest registration matching |
| force-app/main/default/objects/Summit_Events_Registration__c/fields/* | New fields for person accounts, QR codes, and date of birth text |
| force-app/main/default/permissionsets/* | Updated permissions for new fields and objects |
| force-app/main/default/pages/* | API version updates, UI improvements, deprecation markers |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| var error_count = 0; | ||
| var emailReg = /^([a-zA-Z0-9_.\-.'.+])+@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/; | ||
| let error_count = 0; | ||
| let emailReg = /^([a-zA-Z0-9_.\-.'.+])+@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/; |
There was a problem hiding this comment.
Character '.' is repeated in the same character class.
Character '.' is repeated in the same character class.
Create security requirements documentation for SEA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request was automatically generated because an automated merge hit a merge conflict