❄️ Freeze

[HeySreelal]

Tracked in #319.

[HeySreelal]

Keyboard & InlineKeyboard Case

The Keyboard and InlineKeyboard markups were mutable, now with freezed, they're no longer mutable. Hence the following code (which used to work) no longer works.

void main(List<String> args) async {
  final keyboard = Keyboard();
  keyboard.text("Click Me");
  keyboard.requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

---

Solution

1. Update the keyboard variable on each change

Just like how we use copyWith method :)

void main(List<String> args) async {
  var keyboard = Keyboard();
  keyboard = keyboard.text("Click Me");
  keyboard = keyboard.requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

2. Chain it or spread it.

void main(List<String> args) async {
  var keyboard =
      Keyboard().text("Click Me").requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

3. (Untested)

Or may be we can do Keyboard and InlineKeyboard classes to be @unfreezed to make it mutable? May be that works?

• Test this

[im-trisha]

Keyboard & InlineKeyboard Case

The Keyboard and InlineKeyboard markups were mutable, now with freezed, they're no longer mutable. Hence the following code (which used to work) no longer works.

void main(List<String> args) async {
  final keyboard = Keyboard();
  keyboard.text("Click Me");
  keyboard.requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

Solution

1. Update the keyboard variable on each change

Just like how we use copyWith method :)

void main(List<String> args) async {
  var keyboard = Keyboard();
  keyboard = keyboard.text("Click Me");
  keyboard = keyboard.requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

2. Chain it or spread it.

void main(List<String> args) async {
  var keyboard =
      Keyboard().text("Click Me").requestLocation("Send Me Location");

  await api.sendMessage(
    chatId,
    "Use the buttons please.",
    replyMarkup: keyboard,
  );
}

3. (Untested)

Or may be we can do Keyboard and InlineKeyboard classes to be @unfreezed to make it mutable? May be that works?

* [ ]  Test this

The second solution LGTM! It should be the best one, as it doesn't assign the variable again and again (1) and doesn't box/unbox the object every time (3) but just uses the copyWith method :)