search

wefork / wekan

The open-source Trello-like kanban (built with Meteor)
https://wekan.io
MIT License
61 stars 12 forks source link

fix minor bug in Sandstorm username handling #3

Closed dwrensha closed 8 years ago

dwrensha commented 8 years ago

Setting the fullname here can cause the username to be set by this logic before the usual uniqification kicks in. The result is that share-by-powerbox can appear to fail, until the recipient actually visits the grain.

The solution is to adjust the call to Accounts.updateOrCreateFromExternalService() to better match the call in meteor-accounts-sandstrom which gets invoked when the sharing does not go through the powerbox.

dwrensha commented 8 years ago

(Self-merging because this pertains only to Sandstorm.)