## Feature request #### Feature description - According to the current [implementation](https://github.com/ros-planning/navigation2/blob/2214a04a61b1dadc2f2518b29a2237bb3c6f190c/nav2_behavior_tree/include/nav2_behavior_tree/bt_service_node.hpp#L93-L95), the `BtServiceNode` returns `SUCCESS` whenever the service is executed - Even if the service responds with a failure boolean, `BtServiceNode` returns a `SUCCESS` since the service was completed. #### Implementation considerations - This could be solved by adding a `virtual rclcpp::executor::FutureReturnCode check_result(future_result)` that the inherited classess can modify