devise-guests 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 776fbee405a1074096bd13610d3691a1016244e35aa1b25cd36d437077da6d56
-  data.tar.gz: 9d61df8e8b059d679d3a614bc6ffacd72d80a50b8568d7ab03a6aabb2e027611
+  metadata.gz: 7944106e6394d186fa877f614689e47b1059488d6e3d5400ed2eb2204e324f3b
+  data.tar.gz: 817faf49e6a0ad620f2fe35014f1adf1783f3f5177cc7579e4f9bc4cb2872ecd
 SHA512:
-  metadata.gz: 9194038c18a01eaae39f335533d200e4a36acb054af5ac60f9da912b022714162d3240c4c2943bb0d58c6738a9b9e50253dafc078afc689e5055a2e16b541f67
-  data.tar.gz: 4538fc330f09a7678e94b3c4aa21619f93e5469cae3a4159a556c17a14f43960108aa10be69f8f42e0b9ff17ad60e9d71b8e8b8308f1d62bae12ab08471c956a
+  metadata.gz: 98e77fdda01fa6982fb3f2851eb84870ab0a9aeaae60e0bba91c98f55454a705056953738559caad400a162aa7d6d55b3e0b510427f573e094c16eb05d85b181
+  data.tar.gz: 4e9d799034284f840332366ae2d79f87c72a1eeaf1f4526287cd76fd8f50126a51d74749977b0c31479e0c6ab8ff377b9ef2a4e932ec2feb9ada9f6901074e7b

data/.github/workflows/tests.yml

@@ -12,10 +12,39 @@ on:
     branches: [ master ]
   pull_request:
     branches: [ master ]
+    types:
+      - opened
+      - synchronize
+      - reopened
+      - ready_for_review
+      - labeled
+      - unlabeled
 
 jobs:
+  changelog:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - id: read-version
+        run: |
+          echo "::set-output name=VERSION::`cat lib/devise-guests/version.rb | grep -i version | awk '{ print $3 }' | sed -e 's/\"//g'`"
+      - uses: dangoslen/changelog-enforcer@v2.3.1
+        with:
+          skipLabels: 'skip-changelog'
+          expectedLatestVersion: ${{ steps.read-version.outputs.VERSION }}
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.6'
+          bundler-cache: true
+      - run: bundle exec standardrb
   test:
-
+    needs:
+      - changelog
+      - lint
     runs-on: ubuntu-latest
     strategy:
       matrix:

data/CHANGELOG.md

@@ -0,0 +1,25 @@
+# Changelog
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [0.8.1] - 2021-10-26
+### Changed
+- Simplify guest transfer ([#38](https://github.com/cbeer/devise-guests/pull/38))
+
+## [0.8.0] - 2021-10-25
+### Added
+- Allow setting of custom params on guest model ([#31](https://github.com/cbeer/devise-guests/pull/31)) by [@pacso](https://github.com/pacso)
+- Allow deletion of guest records to optionally be skipped ([#32](https://github.com/cbeer/devise-guests/pull/32)) by [@pacso](https://github.com/pacso)
+
+### Changed
+- Switch to SecureRandom.uuid for guest_user_unique_suffix ([#28](https://github.com/cbeer/devise-guests/pull/28)) by [@notch8](https://github.com/notch8)
+- Update combustion gem ([#29](https://github.com/cbeer/devise-guests/pull/29)) by [@notch8](https://github.com/notch8)
+
+### Fixed
+- Defining the logging_in callback when the controller loads ([#34](https://github.com/cbeer/devise-guests/pull/34)) by [@davidkuz](https://github.com/davidkuz)
+
+[Unreleased]: https://github.com/cbeer/devise-guests/compare/v0.8.1...HEAD
+[0.8.1]: https://github.com/cbeer/devise-guests/compare/v0.8.0...v0.8.1
+[0.8.0]: https://github.com/cbeer/devise-guests/compare/v0.7.0...v0.8.0

data/README.md

@@ -1,4 +1,5 @@
 # Devise Guests
+[![Gem Version](https://badge.fury.io/rb/devise-guests.svg)](https://badge.fury.io/rb/devise-guests)
 [![Tests](https://github.com/cbeer/devise-guests/actions/workflows/tests.yml/badge.svg)](https://github.com/cbeer/devise-guests/actions/workflows/tests.yml)
 [![Ruby Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://github.com/testdouble/standard)
 
@@ -41,14 +42,11 @@ guest_user # ( for anonymous users)
 ### Transferring Guest to User on Login
 
 During the login process you may want to transfer things from your guest user to the account the logged into.
-To do so, modify your ApplicationController like so:
+To do so, add the following method to your ApplicationController:
 
 ```ruby
-define_callbacks :logging_in_user
-set_callback :logging_in_user, :before, :transfer_guest_to_current_user
-
 private
-def transfer_guest_to_current_user
+def transfer_guest_to_user
   # At this point you have access to:
   #   * current_user - the user they've just logged in as
   #   * guest_user - the guest user they were previously identified by
@@ -75,6 +73,26 @@ def guest_user_params
 end
 ```
 
+### Non-standard authentication keys
+
+By default Devise will use `:email` as the authentication key for your model. If for some reason you have modified your
+Devise config to use an alternative attribute (such as `:phone_number`) you will need to provide a method to generate
+the value of this attribute for any guest users which are created.
+
+Sticking with the `:phone_number` example, you should define the following method in your `application_controller.rb`:
+
+```ruby
+private
+def guest_phone_number_authentication_key(key)
+  key &&= nil unless key.to_s.match(/^guest/)
+  key ||= "guest_447" + 9.times.map { SecureRandom.rand(0..9) }.join
+end
+```
+
+Validations are skipped when creating guest users, but if you need to rely on future modifications to the guest record
+passing validations, then you should ensure that this default value for guests is generated in such a way as to be
+valid.
+
 ### Prevent deletion of guest records
 
 By default, when signing in from a guest account to an authenticated account, the guest user is destroyed. You have an

data/lib/devise-guests/controllers/helpers.rb

@@ -74,6 +74,9 @@ module DeviseGuests::Controllers
           false
         end
 
+        def transfer_guest_to_#{mapping}
+        end
+
       METHODS
 
       ActiveSupport.on_load(:action_controller) do
@@ -81,6 +84,7 @@ module DeviseGuests::Controllers
           helper_method "guest_#{mapping}", "current_or_guest_#{mapping}"
         end
         define_callbacks "logging_in_#{mapping}"
+        set_callback :"logging_in_#{mapping}", :"transfer_guest_to_#{mapping}"
       end
     end
   end

data/lib/devise-guests/version.rb

@@ -1,3 +1,3 @@
 module DeviseGuests
-  VERSION = "0.8.0"
+  VERSION = "0.8.1"
 end

data/spec/controllers/application_controller_spec.rb

@@ -35,6 +35,7 @@ describe ApplicationController, type: :controller do
     allow(@controller).to receive(:session) { {guest_user_id: 123} }
 
     expect(@controller).to receive(:run_callbacks).with(:logging_in_user).and_call_original
+    expect(@controller).to receive(:transfer_guest_to_user).and_call_original
 
     @controller.current_or_guest_user
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: devise-guests
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Chris Beer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-25 00:00:00.000000000 Z
+date: 2021-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: devise
@@ -133,6 +133,7 @@ files:
 - ".github/workflows/tests.yml"
 - ".gitignore"
 - ".standard.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md