- 
  Notifications
 You must be signed in to change notification settings 
- Fork 89
README: Added known issue for SecurityConfiguration/CredentialEncrypted #703
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
Conversation
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.
Now I remember why I didn't review this: I saw that Richard removed the two sections that he added. Maybe I am not understanding this correctly. Am I missing something? If so, I'll return to this review.
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.
Some minor cleanup suggestions.
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
The following list are known issues. (awkward wording) ->Two choices: In the following list are known issues. or The following list contains known issues.
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
I would suggest code font for discoverDomain , cd(), and ls().
12.2.1 -> 12.2.1.0
discover tool -> Discover Domain Tool
wlst -> WLST
the erroneous message -> an error message
Weblogic WLST -> WLST (WebLogic Scripting Tool, WebLogic WLST is redundant)
12.2.1 PSU -> 12.2.1.0 PSU
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
during discover of a 12.2.1 domain -> during discovery of a 12.2.1.0 domain
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
Severe -> SEVERE
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
11G -> 11g
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
log, -> log (no comma needed)
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
createDomain -> createDomain (code font)
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
before -> prior to
configuration, ->configuration (no comma needed)
 
 
 KnownIssues.md
 
 Outdated
 
 
 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.
for the bug number -> for bug number
your WebLogic version -> your WebLogic Server version
Added titles to each issue.
Corrected some general formatting.