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

Decrement BankingProduct #266

Closed
perlboy opened this issue Jul 13, 2020 · 4 comments
Closed

Decrement BankingProduct #266

perlboy opened this issue Jul 13, 2020 · 4 comments

Comments

@perlboy
Copy link

perlboy commented Jul 13, 2020

Description

BankingProduct is specified as payload V3 within the results for GetProducts however from what we can tell there is zero changes to BankingProduct itself. As far as we can tell the last known change on BankingProduct is the addition of cardArt between V1 and V2.

While BankingProductDetail has changed between V2 and V3, the BankingProduct class has not. As a consequence the generic for BankingProduct is unaltered between V2 and V3 (resulting in class casting failures).

Biza.io intends to implement this non-change in Babelfish as a multi-versioned response with the same underlying class of BankingProductV2.

Area Affected

BankingProductV2 and BankingProductV3

Change Proposed

Remove payload version 3 for GetProducts as the payload has not changed

@CDR-API-Stream
Copy link
Collaborator

Thanks @perlboy this issue is being reviewed as part of Maintenance Iteration 4.

@perlboy
Copy link
Author

perlboy commented Sep 10, 2020

I note in the minutes for Maintenance Iteration 4 this change was not supported and "intentional". As a consequence, are we to assume therefore that the payloads for x-v=2 and x-v=3 for Get Products are the same?

@nils-work
Copy link
Member

Related to #311 ?

@perlboy
Copy link
Author

perlboy commented Sep 11, 2020

@nils-work Similar but not the same. #311 is about a payload changing without being incremented, this one is about a payload incrementing without being changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants