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 for the Netherlands #52

Closed
florianbuerger opened this issue Aug 16, 2022 · 1 comment · Fixed by #53
Closed

Support for the Netherlands #52

florianbuerger opened this issue Aug 16, 2022 · 1 comment · Fixed by #53

Comments

@florianbuerger
Copy link

florianbuerger commented Aug 16, 2022

Love the product 👌 Would be great if stores in the Netherlands would be supported. The stores are there, seems only the country is missing. I was able to make it work for some products (tested the Studio & Studio Display) with just these changes:

diff --git a/InventoryWatch/Countries.swift b/InventoryWatch/Countries.swift
index 2f7b59b..986e730 100644
--- a/InventoryWatch/Countries.swift
+++ b/InventoryWatch/Countries.swift
@@ -63,7 +63,8 @@ let Countries: [String: Country] = [
     "FR": Country(name: "France", storePathComponent: "/fr", skuCode: "FN"),
     "IT": Country(name: "Italy", storePathComponent: "/it", skuCode: "T"),
     "JP": Country(name: "Japan", storePathComponent: "/jp", skuCode: "J"),
-    "AT": Country(name: "Austria", storePathComponent: "/at", skuCode: "D")
+    "AT": Country(name: "Austria", storePathComponent: "/at", skuCode: "D"),
+    "NL": Country(name: "Netherlands", storePathComponent: "/nl", skuCode: "FN")
 ];
 
 let OrderedCountries = [
@@ -77,5 +78,6 @@ let OrderedCountries = [
     "FR",
     "IT",
     "JP",
-    "AT"
+    "AT",
+    "NL"
 ]

It doesn't work for MacBooks, and I am not certain what needs to be changed for them.

@worthbak
Copy link
Owner

Fixed in v0.1.2 👍🎉 https://worthbak.github.io/inventory-checker-app/

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

Successfully merging a pull request may close this issue.

2 participants