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

chore(common): CHECKOUT-00 updated links in README.md #2322

Merged
merged 1 commit into from May 13, 2024

Conversation

bc-0dp
Copy link
Collaborator

@bc-0dp bc-0dp commented Jan 12, 2024

What?

updated links to docs resources to camel case

Why?

Links to docs seems to be case sensitive ie.

https://github.com/bigcommerce/checkout-sdk-js/blob/master/docs/classes/checkoutservice.md#executespamcheck
Would 404,

where as,
https://github.com/bigcommerce/checkout-sdk-js/blob/master/docs/classes/CheckoutService.md#executespamcheck
is OK

Testing / Proof

https://github.com/bigcommerce/checkout-sdk-js/blob/master/docs/classes/checkoutservice.md#executespamcheck
Would 404,

where as,
https://github.com/bigcommerce/checkout-sdk-js/blob/master/docs/classes/CheckoutService.md#executespamcheck
is OK

@bigcommerce/team-checkout @bigcommerce/team-payments

@bc-0dp bc-0dp requested a review from a team as a code owner January 12, 2024 13:10
@bc-0dp bc-0dp changed the title Update README.md broken links chore(readme): CHECKOUT-00 for triggering test Jan 18, 2024
@bc-0dp bc-0dp changed the title chore(readme): CHECKOUT-00 for triggering test chore(readme): CHECKOUT-00 updated links in README.md Jan 18, 2024
@bc-0dp bc-0dp changed the title chore(readme): CHECKOUT-00 updated links in README.md chore(common): CHECKOUT-00 updated links in README.md Jan 19, 2024
@animesh1987
Copy link
Contributor

Thanks @bc-0dp 🙇 Can you please fix up the commit message, so we can get this merged.

@animesh1987 animesh1987 merged commit 6176309 into bigcommerce:master May 13, 2024
7 checks passed
serhii-tkachenko pushed a commit to serhii-tkachenko/checkout-sdk-js that referenced this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants