Save guest only if Model includes Confirmable.

cbeer / devise-guests

fake 'em until you make 'em

MIT License

130 stars 31 forks source link

Save guest only if Model includes Confirmable. #16

Closed gpongelli closed 3 years ago

gpongelli commented 9 years ago

Added save only for models that include the optional Confirmable module, and not for all.

pacso commented 3 years ago

I appreciate that this is a pretty old PR. I've just joined as a new maintainer and am reviewing all of the existing PRs and issues.

I can't see why you'd only want to persist the guest user if they were Confirmable? Am going to close this for now, but if there's a good reason to reopen it please let me know.