Set Default TimeZoneId claim when a new user registers

HTBox / allReady

This repo contains the code for allReady, an open-source solution focused on increasing awareness, efficiency and impact of preparedness campaigns as they are delivered by humanitarian and disaster response organizations in local communities.

http://www.htbox.org/projects/allready

MIT License

891 stars 627 forks source link

Set Default TimeZoneId claim when a new user registers #2296

Closed stevejgordon closed 6 years ago

stevejgordon commented 6 years ago

It looks like when register a new user we don't set a default TimeZoneId claim. Since the claim only gets updated if the user then changes it in their My Account page (only updated if the value actually changed from the model originally rendered) it means quite often a user doesn't get a claim for this.

We should set a default for new users since we don't offer this on the registration form.

NRKirby commented 6 years ago

I see claims are set in the Register POST action here and that there is a TimeZoneId claim type in the ClaimTypes class.

A few questions:

What should the TimeZoneId be set to?
Once this is set, where else in the app should this claim be used?
Is there anything else I need to know?

If you could give me a broad idea of what's involved to resolve this issue I will see if I can look at this..

stevejgordon commented 6 years ago

@NRKirby Sorry for the delay.

I've just tried to replicate this and not been able to. I'm therefore going to close this out for now.