Connected rainwater collector

#include <MKRWAN.h>
#include "Ultrasonic.h"
#include "arduino_secrets.h"

Ultrasonic ultrasonic(7, 6); // Trig et Echo

LoRaModem modem;

// Uncomment if using the Murata chip as a module
// LoRaModem modem(Serial1);


// Please enter your sensitive data in the Secret tab or arduino_secrets.h
String appEui = SECRET_APP_EUI;
String appKey = SECRET_APP_KEY;

void setup() {
  // put your setup code here, to run once:
  Serial.begin(9600);
  pinMode(8, OUTPUT);
  pinMode(9, OUTPUT);
  while (!Serial);
  // change this to your regional band (eg. US915, AS923, ...)
  if (!modem.begin(EU868)) {
    Serial.println("Failed to start module");
    while (1) {}
  };
  Serial.print("Your module version is: ");
  Serial.println(modem.version());
  Serial.print("Your device EUI is: ");
  Serial.println(modem.deviceEUI());

  int connected = modem.joinOTAA(appEui, appKey);
  if (!connected) {
    Serial.println("Something went wrong; are you indoor? Move near a window and retry");
    while (1) {}
  }
  // Set poll interval to 60 secs.
  modem.minPollInterval(61);
  // NOTE: independent of this setting, the modem will
  // not allow sending more than one message every 2 minutes,
  // this is enforced by firmware and can not be changed.
}

void loop() {
  Serial.println();
  Serial.println("Enter a message to send to serial");
  Serial.println("(make sure that end-of-line 'NL' is enabled)");
  fflush(stdin);
  String cmd = "";
  while (!Serial.available());
  cmd = Serial.readStringUntil('\n');

  if(cmd == "a"){
    Serial.println(cmd);
    int dist = ultrasonic.Ranging(CM);
    int P1_state = 0;
    int P2_state = 0;
    if (dist <= 10) {
      digitalWrite(9, LOW);
      digitalWrite(8, HIGH);
      P1_state = 1;
      P2_state = 0;
      Serial.println("led 8 high");
      }
    else {
      digitalWrite(9, HIGH);
      digitalWrite(8, LOW);
      P1_state = 0;
      P2_state = 1;
      Serial.println("led 9 high");
      }  
    Serial.print(dist);
    Serial.println(" cm");
    Serial.print("Etat P1: ");
    Serial.println(P1_state);
    Serial.print("Etat P2: ");
    Serial.println(P2_state);

    byte msg[4] ;

    int err;
    modem.beginPacket();
    msg[0] = dist;
    msg[1] = P1_state;
    msg[2] = P2_state;
    msg[3] = 15;
    modem.write(msg, 4);
    err = modem.endPacket(true);
    if (err > 0) {
      Serial.println("Message sent correctly to LoRa!");
    } else {
      Serial.println("Error sending message :(");
      Serial.print("err:");
      Serial.println(err);
      Serial.println("(you may send a limited amount of messages per minute, depending on the signal strength");
      Serial.println("it may vary from 1 message every couple of seconds to 1 message every minute)");
    }
  }
  else if(cmd == "s"){
    Serial.println(cmd);
    int dist = ultrasonic.Ranging(CM);
    int P1_state = 0;
    int P2_state = 0;
    digitalWrite(9, LOW);
    digitalWrite(8, LOW);
    byte msg[4] ;

    int err;
    modem.beginPacket();
    msg[0] = dist;
    msg[1] = P1_state;
    msg[2] = P2_state;
    msg[3] = 15;
    modem.write(msg, 4);
    err = modem.endPacket(true);
    if (err > 0) {
      Serial.println("Message sent correctly to LoRa!");
    } else {
      Serial.println("Error sending message :(");
      Serial.print("err:");
      Serial.println(err);
      Serial.println("(you may send a limited amount of messages per minute, depending on the signal strength");
      Serial.println("it may vary from 1 message every couple of seconds to 1 message every minute)");
    }
    Serial.println("Pompes stoppées");
  }

  delay(61000);
  Serial.println("ok");
  //if (!modem.available()) {
//    Serial.println("No downlink message received at this time.");
//    return;
//  }
//  char rcv[64];
//  int i = 0;
//  while (modem.available()) {
//    rcv[i++] = (char)modem.read();
//  }
//  Serial.print("Received: ");
//  for (unsigned int j = 0; j < i; j++) {
//    Serial.print(rcv[j] >> 4, HEX);
//    Serial.print(rcv[j] & 0xF, HEX);
//    Serial.print(" ");
//  }
//  Serial.println();
}