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

Support URI Encoded Images in Payloads #64

Closed
perlboy opened this issue Dec 11, 2019 · 4 comments
Closed

Support URI Encoded Images in Payloads #64

perlboy opened this issue Dec 11, 2019 · 4 comments

Comments

@perlboy
Copy link

perlboy commented Dec 11, 2019

Description

Following release to the Slate site of V1.1.0 yesterday we have now initiated analysis of changes using the diff previously provided in #56.

BankingProductV2_cardArt has been introduced to support card art as per #3. We are not aware of the specific proposal for structure on this but find a number of deficiencies with implementation notably:

  1. Based on the description, URL Encoded card art does not appear to be supported. This introduces a requirement for all holders to host a card art location and also limits the opportunities available to provide "live customisation" based on consumer environment (ie. detect a Sydney Swans fan and present a dynamic card art for that)
  2. Data Structure within the swagger file does not meet existing data structure norms

Area Affected

BankingProductV2_cardArt

Change Proposed

  1. Support url encoded card art within the existing structure not just a "Link"
  2. Fix naming of the Data Structure, BankingProductCardArt seems like a similar fit
@CDR-API-Stream
Copy link
Collaborator

CDR-API-Stream commented Mar 2, 2020

Hi @perlboy, card art is currently represented as a URIString which supports valid URI conventions. URI encoded images are based on the data URI scheme (https://en.wikipedia.org/wiki/Data_URI_scheme) which is currently allowed with the existing standards so the proposal for non-URL referenced images is supported.

No change is required.

@perlboy
Copy link
Author

perlboy commented Apr 2, 2020

As stated in the original request:

Based on the description, URL Encoded card art does not appear to be supported.

Specifically the description is:
"Link to a PNG, JPG or GIF image with proportions defined by ISO 7810 ID-1 and width no greater than 512 pixels"

URL Encoded values are not "links" and with the interpretation supplied in @CDR-API-Stream's response of "link" allowing data responses, Holders are now permitted to provide binary content for applicationUri, overviewUri, termsUri eligibilityUri feesAndPricingUri bundleUri among many others.

Both requests still stand.

@CDR-API-Stream
Copy link
Collaborator

Hi @perlboy with respect to the description of a "link" for card art, this can be changed to

URI reference to a PNG, JPG or GIF image with proportions defined by ISO 7810 ID-1 and width no greater than 512 pixels. The URI reference may be a link or url-encoded data URI [RFC 2397].

As for changing BankingProductV3_cardArt to BankingProductCardArt there are already other precedents of the former. This change won't be adopted.

@perlboy
Copy link
Author

perlboy commented May 1, 2020

As for changing BankingProductV3_cardArt to BankingProductCardArt there are already other precedents of the former. This change won't be adopted.

There are also precedents of the latter used throughout. The Watson Guidelines recommend upper camel case. What is requested is consistency but since the DSB continues to inconsistently apply it's own conventions I see no value in continuing to prosecute this. Closing ticket.

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

No branches or pull requests

2 participants