Pues el problema esta ahí, le asignas `null` y luego quieres hacer todo lo demás. Para resolverlo tienes que crear una copia o puedes elegir optimizar el código y ahorrarte unas líneas:

1. Versión como tienes el código:
```
crearReserva(reservaEntry: ReservaPendiente): void {
    const reservaCopy = { ...reservaEntry };
    this.reservaPendiente = reservaEntry;
    this.reservaPendiente = null;
    this.reservaService.crear(reservaCopy).subscribe(
      reserva => {
        console.log(reserva);
        this.reservaService.eliminar(reservaCopy.id); 
        this.router.navigate(['/reservas'])
        swal.fire('Nueva reserva', `La reserva ha sido confirmada con éxito!`, 'success')
      }
    );
}
```
Se resuelve creando una shallow copy (copia superficial) para romper la referencia al objeto original:
```
const reservaCopy = { ...reservaEntry };
```
y usando `reservaCopy` para las operaciones

2. Versión mejorada
```
crearReserva(reservaEntry: ReservaPendiente): void {
    this.reservaService.crear({ ...reservaEntry }).subscribe(
      reserva => {
        console.log(reserva);
        this.reservaService.eliminar(reservaEntry.id);
        this.router.navigate(['/reservas']);
        swal.fire('Nueva reserva', `La reserva ha sido confirmada con éxito!`, 'success');
      }
    );
}
```
En lugar de crear y hacer tanta asignación, enviamos directamente la copia.

En base a tu última pregunta, si `console.log(reserva)` te regresa undefined es porque no regresas nada del método `crear`, si es así comparte el código para seguirte ayudando. 

Como último consejo, puedes crear un método en el servicio para manejar directamente las alertas en lugar de manejarlas a nivel de componente y ahorrarte algo más de código.