Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement a user merging mechanism #4

Open
koniiiik opened this issue Sep 24, 2014 · 6 comments
Open

Implement a user merging mechanism #4

koniiiik opened this issue Sep 24, 2014 · 6 comments

Comments

@koniiiik
Copy link
Member

No description provided.

@mhozza
Copy link
Member

mhozza commented Apr 25, 2016

Toto už máme implementované v trojsten webe

@koniiiik
Copy link
Member Author

On Mon, Apr 25, 2016 at 12:31:30AM -0700, Michal Hozza wrote:

Toto už máme implementované v trojsten webe

Indeed, akurát som nemal veľmi čas sa na to pozrieť do hĺbky. Tak či
tak si myslím, že by malo význam takúto feature naimplementovať tu
(alebo prebrať z trojstenu), keďže moja ambícia bola z tohto spraviť
one-stop-shop pre správu userov.

@mhozza
Copy link
Member

mhozza commented Apr 25, 2016

Tak či
tak si myslím, že by malo význam takúto feature naimplementovať tu

Ja si myslím, že väčší význam by malo mať z toho samostatnú appku. Akurát momentálne je to trochu naviazané na náš user model s user properties, ale to sa da ľahko fixnúť.

@koniiiik
Copy link
Member Author

On Mon, Apr 25, 2016 at 12:44:59AM -0700, Michal Hozza wrote:

Ja si myslím, že väčší význam by malo mať z toho samostatnú appku.
Akurát momentálne je to trochu naviazané na náš user model s user
properties, ale to sa da ľahko fixnúť.

Niekedy si snáď nájdem čas pozrieť sa na to, čo si naimplementoval,
prípadne mi niekedy v lete vysvetlíš pri pive, ako to funguje. (-;
V princípe mi je jedno, či z toho bude samostatný package, alebo to
bude zamontované tu, ale ideálne by bolo niečo, čo je rozumne
zaintegrované to social login workflowu nejakou PSA pipeline funkciou.

@mhozza
Copy link
Member

mhozza commented Apr 25, 2016

Aha, už chápem primárny dôvod,prečo si to tu chcel mať.
Ja to možno v rámci rozbitia trojstenwebu na veľa samostatných balíčkov prerobím na samostatnú appku, tak aby to bolo ľahko integrovatelne do čohokoľvek. Plus by som tam chcel dorobiť všeobecný model merging, čo použijeme potom na školy.
Ale keď na to budeš mať čas skôr ako ja, kľudne sa toho chyť.

@koniiiik
Copy link
Member Author

On Mon, Apr 25, 2016 at 01:05:03AM -0700, Michal Hozza wrote:

Ale keď na to budeš mať čas skôr ako ja, kľudne sa toho chyť.

Ha.

Uvidíme, aký bude nový job v lete, ak budem mať viac priestoru na
dýchanie, možno sa konečne zvládnem pustiť do tých vecí, čo tu mám
naplánované.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants