-
Notifications
You must be signed in to change notification settings - Fork 80
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
base: master
Are you sure you want to change the base?
Conversation
aksarben09
commented
Feb 13, 2018
- withTracker should be used as withTracker(() => {})()
- see https://guide.meteor.com/react.html#using-withTracker
* withTracker should be used as withTracker(() => {})() * see https://guide.meteor.com/react.html#using-withTracker
Line #1025: Accounts.ui.LoginForm = LoginForm; |
I also needed to npm install uuid and add an import for it. |
Can this be merged? |
Seriously, just merge this already and increase the version, then publish so project no longer need to manage a local package just for this! |
I'm going to guess this repo is dead at this point if this still hasn't been merged in? |
If anyone is really annoyed with manually copying the package to their repo, I've forked and republished the package with this change as |
@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. What do I need to do to fix this? Thanks. |
@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 Thanks man, it worked!!! Just a minor correction from the code above.
|