Skip to content

Feed Helper class added#10

Merged
lyubo-slavilov merged 5 commits into
double-break:masterfrom
MST1122:master
Jan 15, 2021
Merged

Feed Helper class added#10
lyubo-slavilov merged 5 commits into
double-break:masterfrom
MST1122:master

Conversation

@MST1122

@MST1122 MST1122 commented Jan 12, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@lyubo-slavilov lyubo-slavilov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It seems good.
@MST1122 please address the comments below, and provide example usage in the README file.

Otherwise everithing looks great.

Comment thread src/Api/Feeds.php Outdated
* The Selling Partner API for Feeds lets you upload data to Amazon on behalf of a selling partner.
*/
namespace DoubleBreak\Spapi\Api;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove this change from the pull request.

Comment thread src/ASECryptoStream.php
@@ -0,0 +1,86 @@
<?php

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please provide your authorship details or original authors and source of the code (if any)

This was linked to issues Jan 15, 2021
@lyubo-slavilov lyubo-slavilov merged commit 540a16e into double-break:master Jan 15, 2021
@lyubo-slavilov lyubo-slavilov added good first issue Good for newcomers enhancement New feature or request labels Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data decryption upload feed

2 participants