Observable dies with skipWhile operator

[syedfahimabrar]

Which @ngneat/elf-* package(s) are the source of the bug?

requests-cache

Is this a regression?

No

Description

In the documentation the skipWhile operator has been described as below:

In addition to value, you can pass in the same object a returnSource which will be returned by the operator if the request is cached. The default return value is EMPTY observable.

todos.service.ts
import { skipWhileTodosCached, setTodos } from './todos.repository';

export function fetchTodos() {
  return http.get(todosUrl).pipe(
    tap((todos) => setTodos(todos)),
    skipWhileTodosCached('todos', { returnSource: of([]) })
  );
}

but when i don't provide any return source then the observable dies. I don't want to pass an empty array either. I want the cached value in the next pipe.

export function fetchTodos() {
  return http.get(todosUrl).pipe(
    tap((todos) => setTodos(todos)),
    skipWhileTodosCached('todos'),
    tap(todos => console.log(todos)
  );
}

i want to print the cached todos with tap like the above mentioned code.

Please provide a link to a minimal reproduction of the bug

No response

Please provide the exception or error you saw

No response

Please provide the environment you discovered this bug in

angular

Anything else?

No response

Do you want to create a pull request?

No

[NetanelBasal]

That's not the purpose of this. You need to query the store if you want the value.

[NetanelBasal]

If you have some good reason for this you can probably do something like this:

export function fetchTodos() {
  return http.get(todosUrl).pipe(
    tap((todos) => setTodos(todos)),
    skipWhileTodosCached('todos', { returnSource: of([]).pipe(map(() => repo.getTodos())) })
  );
}

[syedfahimabrar]

I want to get todos by the cache key, not all todos.

[NetanelBasal]

You'll need to maintain yourself or maybe use the pagination plugin, and each "page" will be the cache key.