-
Notifications
You must be signed in to change notification settings - Fork 262
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
Added DismissRecommendation example. #9
Conversation
086acc0
to
2e4312d
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.
Some minor comment suggestions, otherwise LGTM
2e4312d
to
04dcfe1
Compare
class DismissRecommendation | ||
{ | ||
const CUSTOMER_ID = 'INSERT_CUSTOMER_ID_HERE'; | ||
// Recommendation ID is the last alphanumeric portion of resource name obtained from |
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.
"resource name" -> "the resource name"
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.
Done.
Change-Id: Ic2b030a20a2b59fed5655eea1bf8b897f6a249db
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.
LGTM
No description provided.