square / square/connect-api-specification

Several of the new types are missing their prefixes

Open
#201 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Mustache
Stars
76
Forks
36
Avg merge
3h 14m
Merged PRs (30d)
1

Description

A bunch of the new types have broken references. I believe every case is because they are missing prefix. For example, instead of just Policy it should be CheckoutLocationSettingsPolicy

This commit on my forked version is an easy way to see which ones are broken:
https://github.com/Flash-Order/connect-api-specification/commit/8fa6212acfbef87a42eba6ffe219c2333f8d33fa

Does anyone at Square even care though? I've posted about stuff like this before and don't hear anything back.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the broken type references in the OpenAPI specification with commit 8fa6212acfbef87a42eba6ffe219c2333f8d33fa. Check each reported reference and update missing prefixes so names such as Policy resolve to their full type names; done means the affected references are no longer broken.

Written by the indexing model from the issue text.

Assessment

Tech stack
openapi
Domain
api
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.