Ability to save and load calculator states into two slots #37

Merged
MantashaNoyela merged 4 commits from MantashaNoyela/26 into main 2025-02-25 23:28:55 -08:00
Showing only changes of commit cb44965e30 - Show all commits

View File

@ -1,17 +1,39 @@
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import React, { useState } from "react";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import { ScrollView, Text, Alert, Button, View, StyleSheet } from "react-native";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import React, { useState, useEffect } from "react";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import { ScrollView, Alert, Button } from "react-native";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import PlayerSelector from "@/components/PlayerSelector";
import BuyInSelector from "@/components/BuyInSelector";
import ChipsSelector from "@/components/ChipsSelector";
import ChipDistributionSummary from "@/components/ChipDistributionSummary";
import { saveState, loadState } from "../components/CalculatorState";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import ChipDetection from "@/components/ChipDetection";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
import { saveState, loadState } from "@/components/CalculatorState";
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const IndexScreen = () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
export enum COLORS {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
"white",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
"red",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
"green",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
"blue",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
"black",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
}
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const IndexScreen: React.FC = () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const [playerCount, setPlayerCount] = useState(2);
const [buyInAmount, setBuyInAmount] = useState<number | null>(null);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const [buyInAmount, setBuyInAmount] = useState<number>(20);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const [numberOfChips, setNumberOfChips] = useState<number>(5);
const [totalChipsCount, setTotalChipsCount] = useState<number[]>([]);
useEffect(() => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const loadSavedState = async () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const savedState = await loadState("SLOT1"); // Default loading from SLOT1
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
if (savedState) {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
setPlayerCount(savedState.playerCount);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
setBuyInAmount(savedState.buyInAmount);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
setNumberOfChips(savedState.numberOfChips);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
setTotalChipsCount(savedState.totalChipsCount);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
}
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
};
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
loadSavedState();
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
}, []);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const handleSave = async (slot: "SLOT1" | "SLOT2") => {
if (buyInAmount === null) {
Alert.alert("Error", "Please select a valid buy-in amount");
@ -35,9 +57,13 @@ const IndexScreen = () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
}
};
const updateChipCount = (chipData: { [color: string]: number }) => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
const chipCountArray = Object.values(chipData);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
setTotalChipsCount(chipCountArray);
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
};
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
return (
<View style={styles.container}>
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
<ScrollView contentContainerStyle={{ padding: 20, flexGrow: 1 }}>
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
<ScrollView contentContainerStyle={{ padding: 20, flexGrow: 1 }}>
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
<PlayerSelector playerCount={playerCount} setPlayerCount={setPlayerCount} />
<BuyInSelector setBuyInAmount={setBuyInAmount} />
<ChipDetection updateChipCount={updateChipCount} />
@ -54,21 +80,10 @@ const IndexScreen = () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
/>
<Button title="Save to Slot 1" onPress={() => handleSave("SLOT1")} disabled={buyInAmount === null} />
<Button title="Save to Slot 2" onPress={() => handleSave("SLOT2")} disabled={buyInAmount === null} />
<Button title="Load from Slot 1" onPress={() => handleLoad("SLOT1")} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
<Button title="Load from Slot 2" onPress={() => handleLoad("SLOT2")} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
</ScrollView>
</View>
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
);
};
const styles = StyleSheet.create({
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
container: {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
flex: 1,
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
},
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
title: {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
fontSize: 24,
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
marginBottom: 30,
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
marginTop: 50,
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
textAlign: "center",
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
},
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
});
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
export default IndexScreen;

djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace

  • This title "Poker chips helper" was removed
  • The default state of buyInAmount near the top was changed to 20 from null

Can you fix these up?

It appears that when resolving the merge conflicts, there may have been some regressions. That is to say, these things were changed and committed before you started work, but then undone here. Things I notice on quick glace - This title "Poker chips helper" was removed - The default state of `buyInAmount` near the top was changed to `20` from `null` Can you fix these up?
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
djwesty commented 2025-02-25 21:26:34 -08:00 (Migrated from github.com)
Review

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar.

Nothing to do for now, but keep a mental note

These buttons at the end are ok and satisfy the issue. In the next sprint, we should explore moving them to the title bar and maybe replacing them with an icon or something similar. Nothing to do for now, but keep a mental note
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:03:04 -08:00 (Migrated from github.com)
Review

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.

As I encountered the conflict problem, I removed the folder from my laptop and cloned it again. But I used the code which I had written way before that. May be that's why these problem occured.
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:04:33 -08:00 (Migrated from github.com)
Review

got it

got it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it
MantashaNoyela commented 2025-02-25 22:21:05 -08:00 (Migrated from github.com)
Review

I have changed it

I have changed it