accounts-ui icon indicating copy to clipboard operation
accounts-ui copied to clipboard

Fix call to withTracker

Open aksarben09 opened this issue 7 years ago • 9 comments

  • withTracker should be used as withTracker(() => {})()
  • see https://guide.meteor.com/react.html#using-withTracker

aksarben09 avatar Feb 13 '18 18:02 aksarben09

Line #1025: Accounts.ui.LoginForm = LoginForm; Line #1034: Accounts.ui.LoginForm = LoginFormContainer;

https://github.com/studiointeract/accounts-ui/commit/c8d7653fc0307055b4b019fae7314b8b14f192dc#diff-2b53cd3f80c3586d5e8e2177a6a53728

msteinmn avatar Feb 14 '18 01:02 msteinmn

I also needed to npm install uuid and add an import for it. and change instances of "Meteor.uuid()" to "uuid()"

Kiel-H-Byrne avatar Mar 31 '18 20:03 Kiel-H-Byrne

Can this be merged?

Floriferous avatar Apr 12 '18 15:04 Floriferous

Seriously, just merge this already and increase the version, then publish so project no longer need to manage a local package just for this!

yanickrochon avatar Jul 31 '18 16:07 yanickrochon

I'm going to guess this repo is dead at this point if this still hasn't been merged in?

firrae avatar Sep 06 '18 14:09 firrae

If anyone is really annoyed with manually copying the package to their repo, I've forked and republished the package with this change as epotek:accounts-ui, feel free to use it.

Floriferous avatar Oct 09 '18 09:10 Floriferous

@Floriferous I've tried your forked package and meteor-accounts-t9n (npm), but I'm getting >signIn<, >signUp<,>forgotPassword< etc.. buttons instead of the normal texts.

meteor-login-screenshot from 2018-11-07 10-07-30

What do I need to do to fix this? Thanks.

kurapikats avatar Nov 07 '18 01:11 kurapikats

@kurapikats Here's how to setup basic translation:

import React from 'react';
import { Accounts } from 'meteor/epotek:accounts-ui';
import { T9n } from 'meteor-accounts-t9n';
import { en } from 'meteor-accounts-t9n/build/en';

T9n.map('en', en);
T9n.setLanguage('en');

const App = () => (
  <div>
    <Accounts.ui.LoginForm />
  </div>
);

export default App;

Floriferous avatar Nov 07 '18 07:11 Floriferous

@Floriferous Thanks man, it worked!!!

Just a minor correction from the code above.

import en from 'meteor-accounts-t9n/build/en';

kurapikats avatar Nov 07 '18 12:11 kurapikats