Skip to content

Commit

Permalink
No cancel from negotiation page during re-signing, because that behav…
Browse files Browse the repository at this point in the history
…ior is unintuitive. Only cancel negotiation when proceeding to Free Agency, or when clicking X from Re-sign Players
  • Loading branch information
dumbmatter committed Nov 9, 2024
1 parent 7dea047 commit 147a77b
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 6 deletions.
2 changes: 2 additions & 0 deletions TODO
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
game notes, like player/team notes https://discord.com/channels/290013534023057409/331882115119448065/1304735855042232332

too many non-shooting fouls in late game situations when defense is in the bonus - should be rare when tied or up 1-2 points
- https://discord.com/channels/290013534023057409/331882115119448065/1304329744774336522

Expand Down
20 changes: 14 additions & 6 deletions src/ui/views/Negotiation.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -164,12 +164,20 @@ const Negotiation = ({
</div>
</div>

<button
className="btn btn-danger mt-3"
onClick={() => cancel(player.pid)}
>
Can't reach a deal? End negotiation
</button>
<div className="mt-3">
{resigning ? (
<a
className="btn btn-secondary"
href={helpers.leagueUrl(["negotiation"])}
>
Return to Re-Sign Players page
</a>
) : (
<button className="btn btn-danger" onClick={() => cancel(player.pid)}>
Can't reach a deal? End negotiation
</button>
)}
</div>
</>
);
};
Expand Down

0 comments on commit 147a77b

Please sign in to comment.