-
Notifications
You must be signed in to change notification settings - Fork 8
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
Conversation
…ate/edit/view qa directories. May require some tweaking during integration, but this should be a good stepping stone.
Hmmm, not sure why "templateChooserCommand" test is failing. |
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. |
…nstead of INTERFACES.
@@ -1,81 +1,110 @@ | |||
<!doctype html> | |||
<html lang="en"> | |||
<head> |
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.
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. |
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.
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 = [ |
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.
Just a hard-coded list for now.
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.
Approach looks good to me overall! Certainly in line with what we did for landing page template selection.
May require some tweaking during integration, but this should be a good stepping stone.
@sfdctaka @khawkins @maliroteh-sf