BAM-22413 Update publisher reporting docs #188
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
nattaylor
reviewed
Sep 21, 2023
kavya315
force-pushed
the
BAM-22413
branch
3 times, most recently
from
fdfff70
to
a816fca
Compare
nattaylor
reviewed
Sep 22, 2023
nattaylor
reviewed
Sep 22, 2023
Co-authored-by: Nat Taylor <nattaylor@gmail.com>
nattaylor
reviewed
Sep 22, 2023
Co-authored-by: Nat Taylor <nattaylor@gmail.com>
nattaylor
approved these changes
Sep 22, 2023
hkumar4
reviewed
Sep 22, 2023
| To gain a comprehensive view of returns, we recommend aggregating the values for each unique combination of | ||
| checkout_id, merchant and product_id (denoted by the order_product_id column) across all return files. | ||
|
|
||
| Since this report is designed to be used to complement the data in the orders reports, the column 'order_product_id' |
There was a problem hiding this comment.
Should it be a combination of checkout_id and order_product_id to join
There was a problem hiding this comment.
The event_product_id is a uniquely generated id for each entry in the events_products table so it should be unique to the combination of merchant+checkout_id+product_id.
Co-authored-by: Nat Taylor <nattaylor@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add returns reporting information
Update introduction with all potential locations for report sharing
Add what to expect sections
Add example queries