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

Fix call to withTracker #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aksarben09
Copy link

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

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

c8d7653#diff-2b53cd3f80c3586d5e8e2177a6a53728

@Kiel-H-Byrne
Copy link

Kiel-H-Byrne commented Mar 31, 2018

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

@Floriferous
Copy link

Can this be merged?

@yanickrochon
Copy link

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

@firrae
Copy link

firrae commented Sep 6, 2018

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

@Floriferous
Copy link

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.

@kurapikats
Copy link

kurapikats commented Nov 7, 2018

@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.

@Floriferous
Copy link

@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;

@kurapikats
Copy link

@Floriferous Thanks man, it worked!!!

Just a minor correction from the code above.

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

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

Successfully merging this pull request may close these issues.

None yet

8 participants