[fix bug 979203] Clean up external account data #845

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@jpotts
Contributor
jpotts commented Mar 6, 2014

No description provided.

@glogiotatidis glogiotatidis commented on the diff Mar 12, 2014
...ns/users/migrations/0054_cleanup_external_accounts.py
+ Q(identifier__startswith='http') & (
+ Q(type=self.TYPE_AMO) |
+ Q(type=self.TYPE_BMO) |
+ Q(type=self.TYPE_BITBUCKET) |
+ Q(type=self.TYPE_GITHUB) |
+ Q(type=self.TYPE_MDN) |
+ Q(type=self.TYPE_FACEBOOK) |
+ Q(type=self.TYPE_SLIDESHARE)
+ )
+ )
+
+ for res in results:
+ url = self.ACCOUNT_TYPES[res.type].get('url')
+
+ if url:
+ url_pattern_re = re.sub('{identifier}', '(.*)', url)
glogiotatidis
glogiotatidis Mar 12, 2014 Member

Changing .* to .+ as we need at least on char and merging

Member

r+ this is merged! and I'm sooo happy. Thanks @jpotts 🤘

@jpotts jpotts deleted the jpotts:fix_979203 branch Mar 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment