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
HPCC-21702 New Zap options added to ECL Watch doc #12368
HPCC-21702 New Zap options added to ECL Watch doc #12368
Conversation
https://track.hpccsystems.com/browse/HPCC-21702 |
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.
We should discuss what to do with sensitive info BEFORE the step that says "Press the Apply button."
Something along the lines of:
If your ZAP package might contain sensitive information, use the utility to create a package, then sanitize it, and send manually.
If it doesn't then you can use the email feature to send automatically.
Administrator can configure the email. The (To) Email Address is | ||
also set by the Administrator and can only be changed in the | ||
configuration. The (From) Email Address can be set in the | ||
configuration manager, but can be changed if desired. The Email |
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.
Configuration Manager (s/b title case)
@g-pan back to you |
@JamesDeFabia Revised and Rebased. Please Review. |
@JamesDeFabia Revised and Rebased. |
cd83d09
to
5ac7e32
Compare
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.
only one minor comment inline
Subject is required, but the Email Body is optional.</para> | ||
|
||
<para>Press the <emphasis role="bold">Apply </emphasis>button when | ||
all the dialog fields are filled. At that point if you checked the |
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.
...all the dialog fields are filled. Should be ...all the dialog fields are completed. (or filled in.)
Automated Smoketest: ✅
|
Signed-off-by: G-Pan <greg.panagiotatos@lexisnexis.com>
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.
Ok to merge
Signed-off-by: G-Pan greg.panagiotatos@lexisnexis.com
@JamesDeFabia please review
@kunalaswani FYI, optionally review
View compiled PDF at http://10.240.32.243/view/Docs%207.0/job/AnotherDocBuild7
Type of change:
Checklist:
Testing:
View Successful Build TEST at http://10.240.32.243/view/Docs%207.0/job/AnotherDocBuild7