-
Notifications
You must be signed in to change notification settings - Fork 893
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
Reddit publishers now appear correctly in publisher panel when account is set to hide from robots #2635
Conversation
@@ -194,9 +194,10 @@ std::string MediaReddit::GetUserId(const std::string& response) { | |||
if (response.empty()) { | |||
return std::string(); | |||
} | |||
|
|||
std::string pattern(ExtractData(response, "hideFromRobots\":", |
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.
could you please use std::string pattern =
and const
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
@@ -194,9 +194,10 @@ std::string MediaReddit::GetUserId(const std::string& response) { | |||
if (response.empty()) { | |||
return std::string(); | |||
} | |||
|
|||
std::string pattern(ExtractData(response, "hideFromRobots\":", | |||
"\"isEmployee\"")); | |||
std::string id(braveledger_media::ExtractData( |
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.
could you please use std::string id =
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
6109edb
to
dc4a7af
Compare
|
||
std::string id(braveledger_media::ExtractData( | ||
response, "hideFromRobots\":false,\"id\":\"t2_", "\"")); | ||
const std::string pattern = ExtractData(response, "hideFromRobots\":", |
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.
nit: we always use braveledger_media:: ExtractData
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.
Fixed
dc4a7af
to
fac91f8
Compare
Fixes brave/brave-browser#4754
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests && npm run test-security
) onnpm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
(use reddit.com/u/jsadler-brave as an example of search indexing turned off)
Reviewer Checklist:
After-merge Checklist:
changes has landed on.