Skip to content
This repository has been archived by the owner on Jun 29, 2021. It is now read-only.

Add a onUpdateUser hook to the accounts-base package #68

Closed
fgm opened this issue Jun 9, 2017 · 5 comments
Closed

Add a onUpdateUser hook to the accounts-base package #68

fgm opened this issue Jun 9, 2017 · 5 comments

Comments

@fgm
Copy link

fgm commented Jun 9, 2017

Migrated from: meteor/meteor#7213

@fgm
Copy link
Author

fgm commented Jun 9, 2017

Just had a look at Meteor 1.6 alpha 3 and this mechanism is still absent.

@zimme
Copy link

zimme commented Jun 12, 2017

@fgm, did you ever look into creating a PR for this? If so, any thoughts on how an implementation would look?

@mitar
Copy link

mitar commented Jun 12, 2017

I think you could get this by having a global server-side observe on services key for the user and then run the update on profile.

@fgm
Copy link
Author

fgm commented Jun 12, 2017

@zimme I didn't because meteor contribution rules mention not to send a PR unless one has been requested by MDG.

@mitar Possibly. At any rate, IMHO since such hooks already exist on login and logout and it's marked with a XXX in the code, it's more about fixing an inconsistency than adding a feature, and it should ideally use the same "hook" mechanism as the other two.

@hwillson
Copy link
Contributor

hwillson commented Nov 2, 2017

Addressed by PR meteor/meteor#9042 - thanks!

@hwillson hwillson closed this as completed Nov 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants