jekyll-spree-client 0.1.6 → 0.1.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec972940eca59000e0a3ba7bd764b84531c872e39044f85f2e2a4c4ec61c9ca8
-  data.tar.gz: 3be5a5a3d475b392974c667a2c9b15bad9293e63115b425c7b4f300afd8d1eeb
+  metadata.gz: 57f0f9a9615cb8ecd401ff1c4224b1c246b33489809187f1dacd99bc8efdaba0
+  data.tar.gz: f9279e9889b35a6e7ef42013ca44590796f1530178973d99da5b30d4bb6adcfd
 SHA512:
-  metadata.gz: e02ca9264d8a137c65f5b3e0de3d97171a5432ffe5cab698e8249192ffeb50989993869e6fa9527e042630bb48ed8268f2a3b043158a7e0708483624c4e992ac
-  data.tar.gz: d4da262c648e4006926073e86cd884783a746091784cf96f5544611c428c9657611398ad9f1f6c56e9063ef5b12e3c301647c37f0e10a434884ac334d774859d
+  metadata.gz: 9f77636bc5292b48977e0e7d43de5609d3437568c93aaac1d80f3e9c485b98b87c368d2351770a0c6a047129a1ba2c4a4dc136e6b2ee83ecaf22e0243e37b2ed
+  data.tar.gz: 71243959c46fa41db5d1db9e9a3d661e9284688257b6b00f2a278a6c7174562a0cfdfdfd2197500d141571843f1c6af6290877cbcc70e6bb0590b7ded16ebd80

data/README.md

@@ -16,16 +16,6 @@ conversion unless you store prices as strings.
 
 You can use Integer and Floats now!
 
-### Update
-
-The issue is that Spree expects to receive prices in the locale format
-(ie, "200,00") but will return them on Ruby's `to_s` conversion
-("200.0") so we need to reconvert them to our current format and juggle
-between them.
-
-We'll deal with `Integer` for now until we find [a better
-solution](https://github.com/spree/spree/issues/10556).
-
 ## Installation
 
 Add this line to your site's Gemfile:
@@ -77,8 +67,10 @@ Add the required fields in their frontmatter:
 ---
 layout: post
 title: A product
+description: With a description
 sku: A unique ID
 price: 100.00
+cost_price: 50.0
 stock: 100
 weight: 100
 height: 100
@@ -90,18 +82,64 @@ depth: 100
 The layout doesn't matter, but we recommend to use a layout for
 products.
 
+### Layouts
+
+You'll need four layouts and at least a corresponding post, one for each
+step of the buying process.
+
+* `cart` the cart, shows the order details
+* `shipment` shipping and billing addresses, shipping rates apply here
+* `payment` select payment methods
+* `confirmation` order confirmation page
+
+They'll be available on Liquid under the `site.STEP` keys.
+
+### Hooks
+
+If you need to do some data manipulation before and after
+synchronization is done, you can use hooks.  In a `_plugins/spree.rb`
+file:
+
+```ruby
+Jekyll::Hooks.register :spree, :pre_render do |spree|
+  spree.local_products.each do |product|
+    product.data['_description'] = product.data['description']
+    product.data['description'] = product.data['productore']&.data&.dig('title')
+  end
+end
+
+Jekyll::Hooks.register :spree, :post_render do |spree|
+  spree.local_products.each do |product|
+    product.data['description'] = product.data.delete '_description'
+
+    product.data.delete 'description' unless product.data['description']
+  end
+end
+```
+
+This plugin gets the description value from another document (they're
+linked using
+[jekyll-linked-posts](https://0xacab.org/sutty/jekyll/jekyll-linked-posts/),
+it's a real example!), and then recovers it.
+
+* `pre_render` runs just before starting synchronization
+* `post_render` run after synchronization and before writing changes
+
 ### Building the site
 
 During site build, the plugin will communicate with the Spree API and
-create products when they don't exist or update their price, current
-stock and the other values.
+create products when they don't exist or locally update their price,
+current stock and the other values.
 
 Some fields will appear on the frontmatter:
 
 ```yaml
 variant_id: 1
 in_stock: true
-sync_error: 2020-10-10 00:00:00 UTC
+errors:
+  time: 2020-10-10 00:00:00 UTC
+  messages:
+  - Something went wrong
 ```
 
 * `variant_id` is the ID in the Spree database.
@@ -109,11 +147,17 @@ sync_error: 2020-10-10 00:00:00 UTC
 * `in_stock` is true or false according to availability (Spree decides
   this).
 
-* `sync_error` if the synchronization fails there will be a timestamp
-  here.
+* `errors` if the synchronization fails there will be a timestamp and
+  some messages here.
 
 ## Notes
 
+* Synchronization is unidirectional.  If the product doesn't exist on
+  Spree, it's created.  If it does, the Jekyll document is updated.
+
+  We want this to change in the near future but we need to think a way
+  to prevent conflicts.
+
 * All products are created as "master variants" on Spree.  There will be
   no pictures, taxonomies or other values, only basic product
   information.
@@ -126,11 +170,12 @@ sync_error: 2020-10-10 00:00:00 UTC
 * Stock is added to the default stock location, meaning the first active
   one.  The API doesn't show or allows to query the actual default.
 
-* The `price` is in the default currency.
+* `price` and `cost_price` are expressed in Spree's default currency.
 
 * If you install the `jekyll-write-and-commit-changes` plugin it will
-  save the data back to disk when you build with `JEKYLL_ENV=production`
-  environment variable.
+  save the data back to disk when you build.  Using the
+  `JEKYLL_ENV=production` environment variable will also commit changes
+  to git.
 
 * Check the shipping category ID on Spree, since there's no API for it,
   you need to provide it manually, default is `1`.

data/lib/jekyll-spree-client.rb

@@ -6,7 +6,20 @@ require_relative 'jekyll/spree_client'
 # matter.  The SKU needs to be unique, but this plugin doesn't ensure
 # that.  Duplicated SKUs will be silently ignored for now.
 Jekyll::Hooks.register :site, :post_read do |site|
-  next unless ENV['SPREE_API_KEY']
+  # Ordered articles
+  posts = site.site_payload['site']['posts']
+
+  # Make steps available to other articles
+  %w[cart shipment payment confirmation].each do |step|
+    site.config[step] = posts.find do |doc|
+      doc.data['layout'] == step
+    end
+  end
+
+  unless ENV['SPREE_API_KEY']
+    Jekyll.logger.warn "SPREE_API_KEY environment variable missing, skipping syncrhonization"
+    next
+  end
 
   config = site.config['spree']&.transform_keys(&:to_sym) || {}
   config.merge!(site: site, api_key: ENV['SPREE_API_KEY'], spree_url: ENV['SPREE_URL'])

data/lib/jekyll/spree_client.rb

@@ -33,7 +33,7 @@ module Jekyll
     #
     # @return [Array]
     def variant_fields
-      @variant_fields ||= %w[price weight height width depth].freeze
+      @variant_fields ||= %w[price weight height width depth description cost_price].freeze
     end
 
     # Spree Client
@@ -56,6 +56,8 @@ module Jekyll
     end
 
     def sync!
+      Jekyll::Hooks.trigger :spree, :pre_render, self
+
       # The API allows to query several SKUs at the same time, so we send
       # groups of 10 products.
       local_products.each_slice(10) do |products|
@@ -68,13 +70,14 @@ module Jekyll
           products.each do |p|
             mark_with_error p,
               i18n.dig('spree', 'errors', 'api') || "Couldn't obtain variants, the store may be down or API key is incorrect"
-            local_save p.save
           end
 
           next
         end
 
         products.each do |product|
+          # Remove previous errors
+          product.data.delete 'errors'
           # Find the corresponding Spree variant in the response
           variant = response['variants'].find do |v|
             v[sku_field] == product.data[sku_field]
@@ -88,6 +91,12 @@ module Jekyll
           end
         end
       end
+
+      Jekyll::Hooks.trigger :spree, :post_render, self
+
+      local_products.each do |product|
+        local_save product
+      end
     end
 
     private
@@ -152,19 +161,18 @@ module Jekyll
     def update(product, variant)
       # XXX: The API replies with stringified numbers so we convert them
       # back to the type we're using locally.
-      product.data.merge! variant.slice(*variant_fields).map do |k, v|
-        case product.data[k]
-        when Integer then v.to_i
-        when Float then v.to_f
-        else v
-        end
-      end.to_h
+      variant.slice(*variant_fields).each do |k, v|
+        product.data[k] = case product.data[k]
+                            when Integer then v.to_i
+                            when Float then v.to_f
+                            else v
+                          end
+      end
 
       product.data['in_stock'] = variant['in_stock']
       product.data['variant_id'] = variant['id']
-      product.data.delete 'errors'
 
-      local_save product
+      product
     end
 
     # If the product doesn't exist, create it.
@@ -179,16 +187,12 @@ module Jekyll
       unless remote_products.create **new_product
         Jekyll.logger.error "Couldn't create #{product.data['title']}"
         mark_with_error product, error_messages
-        local_save product
 
         return false
       end
 
       product.data['variant_id'] = remote_products.response.dig('master', 'id')
 
-      # Save the variant ID
-      local_save product
-
       # Add the initial stock
       spree.stock_items(id: product.data['variant_id'], stock_location_id: stock_location['id'])
            .stock_movements.create(quantity: product.data['stock'])
@@ -202,7 +206,7 @@ module Jekyll
         errors.map do |error|
           field + ' ' + error
         end
-      end.flatten
+      end&.flatten
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-spree-client
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.11
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-11-30 00:00:00.000000000 Z
+date: 2021-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree-api-client