-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Rebrand Cloud Job Discovery to Cloud Talent Solution and add the V3 code samples #1166
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
Rebrand Cloud Job Discovery to Cloud Talent Solution and add the V3 code samples #1166
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.
Taking a look at this PR, but note that presubmit and jobs tests have failed. Looks like the api isn't properly enabled in the java-docs-samples-testing project, but please look through and make sure.
Sorry, forgot to unify the setup steps. |
new CreateJobRequest().setJob(jobToBeCreated); | ||
|
||
Job jobCreated = talentSolutionClient.projects().jobs().create(DEFAULT_PROJECT_ID, createJobRequest).execute(); | ||
//Job jobCreated = ta.jobs().create(createJobRequest).execute(); |
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.
delete commented out line
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
.setSessionId("HashedSessionID") | ||
// Domain of the website where the search is conducted | ||
.setDomain( | ||
"www.google.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.
newline not necessary
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
bout.reset(); | ||
} | ||
|
||
// @Test |
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.
Remove commented out test.
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.
Add them back.
Looks like checkstyle errors are causing Kokoro to still fail. You can run this with the maven target |
Done. |
@dpebot Merge when green |
Okay! I'll merge when all statuses are green and all reviewers approve. |
Thank you very much! |
One question, why it is still "Kokoro CI Expected — Waiting for status to be reported" |
Ah, sorry I need to trigger a test run manually. Starting one now. |
Thank you!
…On Wed, Sep 5, 2018 at 12:54 PM Dane Zeke Liergaard < ***@***.***> wrote:
Merged #1166
<#1166> into
master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1166 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AjXhEFLFO0E0X53ubYdPYOGx7rQ766hcks5uYCwJgaJpZM4VnTDE>
.
--
Thank you very much and have a great day!
Xinyun(Phoenix)
Mobile: 734-276-5889
Email: [email protected]
*All things come to those who wait.*
|
Move the previous samples from cjd_sample/ to v2/ and have a new folder v3/ holding v3 samples.
Please hold the test until our V3 API is public in 8/3/2018.