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

Checks for a list of sobjects and the corresponding create/edit/view qa directories. #50

Merged
merged 8 commits into from
Nov 8, 2023

Conversation

dbreese
Copy link
Collaborator

@dbreese dbreese commented Nov 7, 2023

May require some tweaking during integration, but this should be a good stepping stone.

@sfdctaka @khawkins @maliroteh-sf

…ate/edit/view qa directories.

May require some tweaking during integration, but this should be a good stepping stone.
@dbreese dbreese requested a review from a team as a code owner November 7, 2023 01:56
@dbreese
Copy link
Collaborator Author

dbreese commented Nov 7, 2023

Hmmm, not sure why "templateChooserCommand" test is failing.

@dbreese
Copy link
Collaborator Author

dbreese commented Nov 7, 2023

Appears to possibly be a flapper. I just re-ran the failed test and it passed with no issues, so we can keep an eye on ie.

@@ -1,81 +1,110 @@
<!doctype html>
<html lang="en">
<head>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, most of the changes is just the addition of the scripts in the html. I made mistake of reformatting doc without realizing.

</html>
});

// Wait until all scripts are loaded, before engaging with e.g.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the meat of the change here -- makes a call to new function and dynamically generates table rows.

type: 'getQuickActionStatus',
action: async (_panel, data, callback) => {
// TODO: Hook this up to function from W-14439417 that parses landing_page.json.
const sobjects = [
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a hard-coded list for now.

@dbreese dbreese changed the title @W-14442225 - Checks for a list of sobjects and the corresponding create/edit/view qa directories. Checks for a list of sobjects and the corresponding create/edit/view qa directories. Nov 7, 2023
Copy link
Collaborator

@khawkins khawkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approach looks good to me overall! Certainly in line with what we did for landing page template selection.

@dbreese dbreese merged commit e9fa2f8 into salesforce:main Nov 8, 2023
9 checks passed
@dbreese dbreese deleted the check_for_qas branch November 8, 2023 15:17
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

2 participants