david/poker-chips-helper
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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
Conversation 1 Commits 4 Files Changed 5 +17 -19
1 changed files with 17 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a3288e2a31 - Show all commits

36
app/index.tsx
View File

@ -38,25 +38,23 @@ const IndexScreen = () => {
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

got it

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

I have changed it

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

I have changed it

I have changed it
return (
<View style={styles.container}>
<ScrollView contentContainerStyle={{ padding: 20, flexGrow: 1 }}>
<Text style={styles.title}>Poker Chip Helper</Text>
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
<PlayerSelector playerCount={playerCount} setPlayerCount={setPlayerCount} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
<BuyInSelector setBuyInAmount={setBuyInAmount} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
<PlayerSelector playerCount={playerCount} setPlayerCount={setPlayerCount} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
<BuyInSelector setBuyInAmount={setBuyInAmount} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
<ChipDetection updateChipCount={updateChipCount} />
djwesty commented 2025-02-25 21:23:25 -08:00 (Migrated from github.com)
Review
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

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

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

I have changed it

I have changed it
</View>
);
};

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

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

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
Copy Link

got it

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

got it

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

I have changed it

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

I have changed it

I have changed it