这是indexloc提供的服务,不要输入任何密码
Skip to content

Rename ContextWrapper to AuthProvider for clarity in AuthContext.jsx #4510

@angelplusultra

Description

@angelplusultra

How are you running AnythingLLM?

All versions

What happened?

The Auth context provider is exported as ContextWrapper, which is not a clear or descriptive name. It should be exported as AuthProvider.

Relevant code references:

export function ContextWrapper(props) {
const localUser = localStorage.getItem(AUTH_USER);
const localAuthToken = localStorage.getItem(AUTH_TOKEN);
const [store, setStore] = useState({
user: localUser ? JSON.parse(localUser) : null,
authToken: localAuthToken ? localAuthToken : null,
});
const [actions] = useState({
updateUser: (user, authToken = "") => {
localStorage.setItem(AUTH_USER, JSON.stringify(user));
localStorage.setItem(AUTH_TOKEN, authToken);
setStore({ user, authToken });
},
unsetUser: () => {
localStorage.removeItem(AUTH_USER);
localStorage.removeItem(AUTH_TOKEN);
localStorage.removeItem(AUTH_TIMESTAMP);
setStore({ user: null, authToken: null });
},
});
return (
<AuthContext.Provider value={{ store, actions }}>
{props.children}
</AuthContext.Provider>
);
}

<ContextWrapper>
<LogoProvider>

Are there known steps to reproduce?

  1. Open frontend/src/AuthContext.jsx.
  2. See that the provider is exported as ContextWrapper.

Metadata

Metadata

Assignees

Labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions