Skip to content

Comments

Minor changes in the /master/dev-docs/bidders/pixfuture.md#3108

Closed
pixfuture-media wants to merge 10 commits intoprebid:masterfrom
PixFuture-Media-Inc:master
Closed

Minor changes in the /master/dev-docs/bidders/pixfuture.md#3108
pixfuture-media wants to merge 10 commits intoprebid:masterfrom
PixFuture-Media-Inc:master

Conversation

@pixfuture-media
Copy link
Contributor

Two parameters pbjs and pbs has been changed in the /master/dev-docs/bidders/pixfuture.md

Copy link
Collaborator

@MartianTribe MartianTribe left a comment

Choose a reason for hiding this comment

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

Hi, minor typo and grammatical error:

The pix_id parameter should be anstringr<<

Should be: The pix_id parameter should be a String

I would have fixed it but I couldn't edit the file.

@pixfuture-media
Copy link
Contributor Author

The minor typo and grammatical error: In the pix_id description has been removed space

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

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

Description still says "should be an stringr"

@pixfuture-media
Copy link
Contributor Author

Misspelling 'stringr' changed to 'string'

@pixfuture-media
Copy link
Contributor Author

We fixed that misspelling.
Question: how will resolve branch conflict? It should be resolve by persona whom have an write-access...
Please let us know about next steps.

bretg added a commit that referenced this pull request Jul 16, 2021
replaces #3108 

In the future, allowing us to modify your fork would make fixing conflicts like this easier.
bretg added a commit that referenced this pull request Jul 16, 2021
replaces #3108 

In the future, allowing us to modify your fork would make fixing conflicts like this easier.
@bretg
Copy link
Contributor

bretg commented Jul 16, 2021

Fixed with #3117 -- that was easier than trying to fix your fork without write perms.

@bretg bretg closed this Jul 16, 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