brodanoel / duix Goto Github PK

View Code? Open in Web Editor NEW

50.0 4.0 5.0 157 KB

A State Manager focused on KISS and Pareto's Principle

License: MIT License

JavaScript 100.00%

state manager statemanager state-management state-manager callbacks redux vuex kiss pareto

duix's People

Contributors

Stargazers

Watchers

Forkers

teddygandon lsmoura cybernetics cuulee fengluandll

duix's Issues

Implement Events

Implement events. For example: "User just logged"

Should be implemented in some way like:

duix.event('USER_JUST_LOGGED'); // not sure if `duix.event` or `duix.dispatch`

duix.listen(‘USER_JUST_LOGGED’, callback);

Importing "index.js" in browser does not work

By importing

import deepEqual from './utils/deepEqual';

The import does not work in browser, because the extension is not set.

Use instead:

import deepEqual from './utils/deepEqual.js';

Add doc

idea

const store = Object.create(null)

export default ((host = Object.create(null), subs = new Map()) => Object.assign(host, {
    emit(event, ...data) {
        if (subs.has(event)) for (const h of subs.get(event)) h.apply(null, data)
    },
    subscribe(event, handler) {
        if (!(handler instanceof Function)) throw new TypeError('subs need functions')
        if (!subs.has(event)) subs.set(event, new Set())
        subs.get(event).add(handler)
        return () => host.unsubscribe(event, handler)
    },
    unsubscribe (event, handler) {
        if (!subs.has(event)) return
        const ls = subs.get(event)
        ls.delete(handler)
        if (!ls.size) subs.delete(event)
    },
    set(key, val) {
        const old = store[key]
        if (old != null && old === val) return
        host.emit(key, store[key] = val, old)
    },
    get: key => store[key]
}))()

DUIX doesn't work in React with Hooks

If you are doing:

useEffect(() => {
  duix.subscribe('user', setUser)
}, []);

The setUser function is going to get old in the following render.
We have to find a way to keep it simple, but fix it.

I think we should not create a useDuix(), because the idea is to keep DUIX as a state manager based on events.

Should have an automated test

Would travis be ok for you?

=> If yes, I can implement it, but you need to enable the integration
If no, tell me which one to use :-)

Option "callMeNow" is not documented

But is it a easy to understand name ?
See a beginning of discussion in #10

Is it still maintained? Yes, it is.

This, it is still maintained. There is just nothing to improve here.

Add TypeScript doc files

Example from Expo

build.d.ts

declare enum BackgroundFetchResult {
    NoData = 1,
    NewData = 2,
    Failed = 3
}
declare enum BackgroundFetchStatus {
    Denied = 1,
    Restricted = 2,
    Available = 3
}
interface BackgroundFetchOptions {
    minimumInterval?: number;
    stopOnTerminate?: boolean;
    startOnBoot?: boolean;
}
export declare function getStatusAsync(): Promise<BackgroundFetchStatus | null>;
export declare function setMinimumIntervalAsync(minimumInterval: number): Promise<void>;
export declare function registerTaskAsync(taskName: string, options?: BackgroundFetchOptions): Promise<void>;
export declare function unregisterTaskAsync(taskName: string): Promise<void>;
export { BackgroundFetchResult as Result, BackgroundFetchStatus as Status, };