jekyll-spree-client 0.1.5 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dd9907ed1dc9502f568dd85c5cebd6c822721632e9e9cf0d6817b5cf7df07e9e
-  data.tar.gz: 727626b4a4beebcd90c0eea5c7bf00bb1b0eb5d6cc137c667fb17153be15c97a
+  metadata.gz: 716e8e0f2ab2c09b987a10a760ed2e4bf36ebb142ff09b1331a03caaa824c57a
+  data.tar.gz: 7f5048e01435bde67988e5d4690e3026aed28ffbe75684cfd3b4526febe0d1af
 SHA512:
-  metadata.gz: d66fb7b5f68bd1f9e36096e8cd3b26b26da41862fc4827b29a830f9c0f5f0784cacf734a0f67ad58e54f46c53fe930783b1911b211039715516690c317888465
-  data.tar.gz: a035f279a9f2874d6cd4ba3e97c5338c99a15b4563c810400b5b3c8859713b9d51e240a5631080900ac656f97567cc842c2fe51f022fed414784c83d5357749c
+  metadata.gz: 2824a7951b2c4e9affb55ba9bee0bc249f4e728db341b2704181a16130ec49274d248110a2c8d072921018ab2f807fc870978fd874891bced7925971560f92aa
+  data.tar.gz: 61cdaaca089e6fdbeeae9dc3be9d904730e0ba322fd653b36fa9b4d979cccda4c0aa618a7e21e46c7b62157c781e6d37f8035bb05f4f35cbb8a5f39d3ed8b370

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,9 +6,22 @@ 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']
 
-  config = site.config['spree']&.transform_keys(&:to_sym)
+  # 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'])
 
   client = Jekyll::SpreeClient.new **config

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'])

metadata

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