-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a few instances of "Go Commerce" with an extra space. I don't have commit access on this repo, so I went the easy way and just made comments. 😈
README.md
Outdated
@@ -1,16 +1,16 @@ | |||
# Netlify Commerce JS | |||
# Go Commerce JS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove space
README.md
Outdated
|
||
This is a JS client library for [Netlify Commerce](https://github.com/netlify/netlify-commerce) API. | ||
This is a JS client library for [Go Commerce](https://github.com/netlify/gocommerce) API. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove space
README.md
Outdated
@@ -80,7 +80,7 @@ commerce.setCountry("USA"); | |||
commerce.setCurrency("USD"); | |||
``` | |||
|
|||
You can use Netlify Commerce JS together with [authlify](https://github.com/netlify/authlify) to let users log in and claim view order history. | |||
You can use Go Commerce JS together with [GoTrue](https://github.com/netlify/gotrue) to let users log in and claim view order history. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove space
Looks good to me! Though I can't fully confirm the code part is good. I assume npm/etc. handles the name change so projects using netlify-commerce-js don't break? |
Authlify needs to be changed to goTrue in the readme |
also src/index.js references |
I am not seeing those @bdougie |
my bad, I was looking at master. This is good |
mostly just a lot of search and replace ~ @biilmann could you check this is right?