Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

Is this Repository active? #84

Open
exapsy opened this issue Oct 4, 2017 · 22 comments
Open

Is this Repository active? #84

exapsy opened this issue Oct 4, 2017 · 22 comments

Comments

@exapsy
Copy link

exapsy commented Oct 4, 2017

I would like to use it in some cases, but if the community and the administrator are inactive I won't do such thing.

As I see the newest issue was on 3rd of August and it wasn't even answered.
The last answered by the Administrator of this repository was on May 16.

So, what is going on?

@mrchief
Copy link

mrchief commented Dec 26, 2017

This is a good library. We all understand that most OSS projects are not bread earners and it can hard to find time for them. I urge the owners to transfer it to the community so that we don't see this go to a waste.

@tonyhb

@exapsy
Copy link
Author

exapsy commented Dec 28, 2017

@mrchief There are other great libraries though. Try https://github.com/erikras/redux-form if you're currently searching for one. Since that one has ... no pulse.

@mrchief
Copy link

mrchief commented Dec 28, 2017

@Eksapsy I use redux-form but that and this one are for completely different purposes. redux-form is a not a state management library unless I missed something.

@amok
Copy link

amok commented Jan 9, 2018

@Eksapsy
going to give jasonmorita/react-redux-ui-state a shot
except some stripped out features like passing a reducer to hoc it looks very similar to redux-ui

UPD
and... it doesn't expose actions (of course there is an option to create bound to dispatch set of actions, but...)

@mrchief
Copy link

mrchief commented Jan 9, 2018

@amok Keep us posted please!

@amok
Copy link

amok commented Jan 9, 2018

@mrchief I finally ended up writing this little bicycle

import { get } from 'lodash'
import { map } from 'redux-data-structures'
import { connect } from 'react-redux'
import { createAction } from 'redux-actions'
import { compose, lifecycle } from 'recompose'

export const STATE_KEY = 'rui'

export const UPDATE = '@@UI/UPDATE'
export const CREATE = '@@UI/CREATE'
export const REMOVE = '@@UI/remove'

const meta = name => ({ name })
const payload = (name, opts) => opts
const toUIState = (f, p) => typeof f === 'function' ? f(p) : f

export const update = createAction(UPDATE, payload, meta)
export const create = createAction(CREATE, payload, meta)
export const remove = createAction(REMOVE, payload, meta)

export const reducer = map({
  keyGetter: action => get(action, 'meta.name'),
  itemModifier: (item, action) => ({ ...item, ...action.payload }),
  addActionTypes: [CREATE],
  changeActionTypes: [UPDATE],
  removeActionTypes: [REMOVE]
})

export const withUIState = ({ name, state }) => compose(
  connect(
    (state, ownProps) => ({
      ui: get(state, `${STATE_KEY}.byId.${name}`) || toUIState(state, ownProps)
    }),
    {
      setUI: vals => update(name, vals),
      createUI: vals => create(name, vals)
    }
  ),
  lifecycle({
    componentDidMount () {
      this.props.createUI(toUIState(state, this.props))
    }
  })
)

(will add "clear on unmount" and other stuff later)

so... nothing to report about :)

@amok
Copy link

amok commented Jan 9, 2018

my hope is that @tonyhb will get back to maintaining this project

@mrchief
Copy link

mrchief commented Jan 10, 2018

@amok I've resisted writing that bicycle in the hope that @tonyhb returns. But I think I'll take ur bike in the short term. :)

@ghost
Copy link

ghost commented Jan 24, 2018

Trying to get this moving again, since it serves my purposes at work.

While waiting on @tonyhb, feel free to check out my fork.

Notable changes in the fork include reviewing/merging the pull requests that have piled up here and removing the run-time dependency on Immutable.JS.

EDIT: Changed link to reflect package name change in npm

@mrchief
Copy link

mrchief commented Jan 24, 2018

@jhorowitz-firedrum Thanks for keeping the flame alive!

I was almost about to get rid of it but looks like I don't have to (since you got rid of ImmutbaleJS as well)!

@hedgerh
Copy link

hedgerh commented Feb 9, 2018

I'm not sure why @tonyhb has gone complete radio silence on this project. It seems like he's really active on github. Not sure why he isn't responding to offers to help maintain the project. I have created my own fork as well, and I'm considering doing a bit of a rewrite of it to make it do some things I need it to do. https://github.com/hedgerh/redux-fui

I'm trying to use it at Kroger. Maybe we could discuss creating an org for a fork? It'd be great if @tonyhb were willing to move this project to an org. Otherwise, I may create a kroger fork on our internal npm or something.

The features we need on my project:

  • Store UI state under a custom key, rather than just state.ui
  • Be able to easily update the UI state by dispatching actions. We have sagas that need to be able to change the UI state via actions. For example, when a user submits a form, close it if the post was successful. Right now, I pass each redux-ui HOC a custom reducer that listens for a RESET_UI action to reset to default UI. Would be interested in seeing an easy way to update UI state like this without custom reducers or needing to pass the ui keys along to my sagas or something.

@mrchief
Copy link

mrchief commented Feb 9, 2018

@hedgerh Take a look at @jhorowitz-firedrum's fork. I think he's unofficially taken over this project. I think creating an org might be a good idea. Everyone forking and going their own ways (they still can if they strongly feel so) doesn't sound like a productive roadmap to me.

I'd love to contribute to this library anyway I can.

@hedgerh
Copy link

hedgerh commented Feb 9, 2018

Yeah I definitely think @jhorowitz-firedrum has put more work into his than mine. My changes are pretty small and could be added on top of his fork. @jhorowitz-firedrum, would you be interested in moving your fork into an org?

also, i don't dislike the package name react-redux-ui-tools, but it's a little lengthy. i named mine redux-fui (f for fork), which isn't the best name either, but your fork can have my fork's name if you'd like. im fine with any name, but just thought i'd offer.

@ghost
Copy link

ghost commented Feb 9, 2018

Sounds good to me @hedgerh, the more help the better :)

Invite to collaborate sent. 👍

As far as the name, while I agree it is lengthy I figure react-redux specific packages should start with react-redux-. I'm open to changing the latter part, but unfortunately react-redux-ui was taken on npm :(

@mrchief
Copy link

mrchief commented Feb 9, 2018

@jhorowitz-firedrum Note that this has nothing to do react. Just like redux has nothing to do with react. And we can ask npm folks to release redux-ui tag as the author is not active. It'll take some time but we may get it.

@ghost
Copy link

ghost commented Feb 9, 2018

@mrchief the package depends on react at runtime; it is essentially a react HOC package with a set of convenience props and a reducer which leverages redux.

@mrchief
Copy link

mrchief commented Feb 10, 2018

Yes you're right. I forgot about the HOC part.

@jasonmorita
Copy link

jasonmorita commented Mar 3, 2018

Hey @amok I randomly ran across your comment here today. My lib is still being used by my former team and whomever else uses it out there in userland, there just haven't been any issues reported lately, but the repo is active (https://www.npmjs.com/package/react-redux-ui-state).

I would love feedback from any of you who have used the package. It solved a lot of problems for us.

If there are missing features, please make a PR. We found this gave us all we needed for our app's needs, which is what it's pretty simple.

Thanks!

@cristian-sima
Copy link

I think it is dead

@mrchief
Copy link

mrchief commented Feb 19, 2019

I think it is dead

Oh well, the world needs a hooks version of this anyway!

@tonyhb
Copy link
Owner

tonyhb commented Feb 19, 2019 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants
@tonyhb @mrchief @amok @jasonmorita @hedgerh @exapsy @cristian-sima and others