Tap deprecated use an observer argument

Warning:

Instead of passing separate callback arguments, use an observer argument. 
Signatures taking separate callback arguments will be removed in v8

Instead of passing separate callback arguments, use an observer argument. tap signature taking separate callback arguments will be removed in v8

The solution is that instead of using the following signature:

      tap(
        (value) => {
         // success
        },
        (error: any) => {
         // error
        }
      )

you need to use the following signature for tap now:

     tap({
        next: (value) => {
         // success
        },
        error: (error: any) => {
         // error
        }
      })