Skip to content
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

ETARGET Bid adapter: add get meta data functionality #6789

Closed
wants to merge 26 commits into from
Closed

ETARGET Bid adapter: add get meta data functionality #6789

wants to merge 26 commits into from

Conversation

etargetse
Copy link
Contributor

@etargetse etargetse commented May 19, 2021

Type of change

  • Bugfix
  • Feature
  • New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?
  • Other

Description of change

  • test parameters for validating bids
{
  bidder: '<bidder name>',
  params: {
    // ...
  }
}

Be sure to test the integration with your adserver using the Hello World sample page.

  • contact email of the adapter’s maintainer
  • official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

Other information

@ChrisHuie ChrisHuie changed the title new functionalities in ETARGET adapter ETARGET Bid adapter: add get meta data functionality May 19, 2021
@ChrisHuie
Copy link
Collaborator

@etargetse looks like you have some linting errors on this pr

@ChrisHuie ChrisHuie self-requested a review May 19, 2021 15:50
@ChrisHuie ChrisHuie self-assigned this May 19, 2021
@patmmccann
Copy link
Collaborator

tagging as taxonomy, likely needs committeee discussion

@etargetse
Copy link
Contributor Author

@ChrisHuie can you please check what is wrong with etargetBidAdapter.md file? circleci faild test but details reveals nothing, just infinite loading window so i have no info whats wrong. Thanks.

@ChrisHuie
Copy link
Collaborator

ChrisHuie commented May 20, 2021

Here are the linting errors I am seeing in CircleCi. Not seeing anything wrong with the md file? We don't really check markdown files with tests just code files. @etargetse

/home/circleci/Prebid.js/modules/etargetBidAdapter.js
27:26 error Missing space before opening brace space-before-blocks
30:26 error A space is required after ',' comma-spacing
30:37 error A space is required after ',' comma-spacing
30:51 error A space is required after ',' comma-spacing
30:68 error A space is required after ',' comma-spacing
30:77 error A space is required after ',' comma-spacing
30:84 error A space is required after ',' comma-spacing
31:5 error Expected space(s) after "try" keyword-spacing
32:7 error Expected space(s) after "for" keyword-spacing
32:27 error Missing space before opening brace space-before-blocks
33:1 error Expected indentation of 8 spaces but found 10 indent
33:11 error Expected space(s) after "if" keyword-spacing
33:14 error Unary word operator 'typeof' must be followed by whitespace space-unary-ops
33:31 error Infix operators must be spaced space-infix-ops
33:42 error Missing space before opening brace space-before-blocks
34:1 error Expected indentation of 10 spaces but found 14 indent
35:1 error Expected indentation of 10 spaces but found 14 indent
36:1 error Expected indentation of 10 spaces but found 14 indent
36:15 error Expected space(s) after "if" keyword-spacing
36:34 error Infix operators must be spaced space-infix-ops
36:36 error Strings must use singlequote quotes
36:54 error Missing space before opening brace space-before-blocks
37:1 error Expected indentation of 12 spaces but found 18 indent
37:19 error Expected space(s) after "if" keyword-spacing
37:43 error Infix operators must be spaced space-infix-ops
37:47 error Missing space before opening brace space-before-blocks
38:1 error Expected indentation of 14 spaces but found 22 indent
38:23 error Expected space(s) after "if" keyword-spacing
38:38 error Missing space before opening brace space-before-blocks
39:1 error Expected indentation of 16 spaces but found 26 indent
40:1 error Expected indentation of 14 spaces but found 22 indent
40:24 error Trailing spaces not allowed no-trailing-spaces
41:1 error Expected indentation of 14 spaces but found 22 indent
42:1 error Expected indentation of 12 spaces but found 18 indent
43:1 error Expected indentation of 10 spaces but found 14 indent
44:1 error Expected indentation of 8 spaces but found 10 indent
49:6 error Expected space(s) before "catch" keyword-spacing
49:6 error Expected space(s) after "catch" keyword-spacing
49:14 error Missing space before opening brace space-before-blocks
87:17 error 'getMetaData' is not defined no-undef

/home/circleci/Prebid.js/test/spec/modules/etargetBidAdapter_spec.js
21:1 error Trailing spaces not allowed no-trailing-spaces

✖ 41 problems (41 errors, 0 warnings)
40 errors, 0 warnings potentially fixable with the --fix option.

@etargetse
Copy link
Contributor Author

etargetse commented May 20, 2021

@ChrisHuie can you explain for me errors which are now shown in circleci ? Thanks .

@ChrisHuie
Copy link
Collaborator

@etargetse it looks like CircleCi is currently experiencing an outage so tests are hanging up at the moment.

@etargetse
Copy link
Contributor Author

@ChrisHuie circleci still fails on Firefox 73, can you explain why? from the log it is not clear. Thanks.

@ChrisHuie
Copy link
Collaborator

ChrisHuie commented May 24, 2021

@etargetse I'm not 100% sure but think it is either unsupported strings or data types being passed in IE on the end-to-end tests or there is latency being introduced from scrapping the meta data that is impacting different types of other adapters

@etargetse
Copy link
Contributor Author

etargetse commented May 24, 2021

@ChrisHuie so when i dont know how to fix problem or even if there is problem, what should i do next? can we pass test manualy without circleci?

function getMetaData() {
var mts = {};
var hmetas = document.getElementsByTagName('meta');
var wnames = ['title', 'og:title', 'description', 'og:description', 'og:url', 'base', 'keywords'];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you look in the ortb2 object for this stuff as well?

Publishers or a prebid module might be setting it directly

function setKeywords() {

or via methods in #5795

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for recommendation, but this is not solving my problem. With or without my new function "getMetaData" circleci is failing tests.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm aware, this is something else you should fix

@ChrisHuie
Copy link
Collaborator

ChrisHuie commented May 26, 2021

@etargetse I messed around with this a bit and think pulling in upstream commits from master might automatically fix the issue. This pr is a bit behind the main branch. The initial commits that passed testing were from 6 months ago so there is a bit of a gap from when this was initially forked to now.

@etargetse
Copy link
Contributor Author

@ChrisHuie ok i will close this pull request and try new one on master, thanks

@etargetse etargetse closed this May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants