Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[bug] `LISTENER_RESPONSE` does not update `ordered`

See original GitHub issue

Do you want to request a feature or report a bug?
bug

What is the current behavior? When received action @@reduxFirestore/LISTENER_RESPONSE, the doc was already loaded and storeAs is set, only data got updated, not ordered.

What is the expected behavior? Expect both ordered and data to be updated.

Which versions of dependencies, and which browser are affected by this issue? Did this work in previous versions or setups?

redux-firestore@^0.5:
  version "0.5.0"
react-redux-firebase@^2.1:
  version "2.1.0"

Steps to reproduce and if possible a minimal demo of the problem via codesandbox or similar. Listener setup:

collection(pin): "discussions"
doc(pin): "5aed4e0740133e2af516db07"
storeAs(pin): "discussionById_5aed4e0740133e2af516db07"

Received action:

{
  type: '@@reduxFirestore/LISTENER_RESPONSE',
  meta: {
    collection: 'discussions',
    doc: '5aed4e0740133e2af516db07',
    storeAs: 'discussionById_5aed4e0740133e2af516db07'
  },
  payload: {
    data: {
      '5aed4e0740133e2af516db07': {
        text: '123',
        id: '5aed4e0740133e2af516db07',
      }
    },
    ordered: [
      {
        id: '5aed4e0740133e2af516db07',
        text: '123',
      }
    ]
  },
  merge: {}
}

RFF config:

  userProfile: 'users',
  attachAuthIsReady: true,
  presence: false,
  useFirestoreForProfile: true,
  logListenerError: true,
  oneListenerPerPath: true,
  enableLogging: true,

Failing test

      it.only('updates doc under storeAs', () => {
        action = {
          type: actionTypes.LISTENER_RESPONSE,
          meta: {
            collection: 'testing',
            doc: '123abc',
            storeAs: 'pathName',
          },
          payload: {
            ordered: [
              {
                content: 'new',
              },
            ],
          },
          merge: {},
        };

        state = {
          pathName: [
            {
              id: '123abc',
              content: 'old',
            },
          ],
        };
        expect(orderedReducer(state, action)).to.have.nested.property(
          `pathName.0.content`,
          'new',
        );
      });