accounts-ui
accounts-ui copied to clipboard
Fix call to withTracker
- withTracker should be used as withTracker(() => {})()
- see https://guide.meteor.com/react.html#using-withTracker
Line #1025: Accounts.ui.LoginForm = LoginForm; Line #1034: Accounts.ui.LoginForm = LoginFormContainer;
https://github.com/studiointeract/accounts-ui/commit/c8d7653fc0307055b4b019fae7314b8b14f192dc#diff-2b53cd3f80c3586d5e8e2177a6a53728
I also needed to npm install uuid and add an import for it. and change instances of "Meteor.uuid()" to "uuid()"
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 epotek:accounts-ui
, feel free to use it.
@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.
import en from 'meteor-accounts-t9n/build/en';