Skip to content

feat(golf): add My Bag section sourced from static JSON - #105

Merged
kenjdavidson merged 2 commits into
mainfrom
copilot/add-golf-bag-section
Mar 16, 2026
Merged

feat(golf): add My Bag section sourced from static JSON#105
kenjdavidson merged 2 commits into
mainfrom
copilot/add-golf-bag-section

Conversation

Copilot AI commented Mar 16, 2026

Copy link
Copy Markdown
Contributor
  • Create src/data/golf-bag.json with club data (type, name, model, avgDistance)
  • Create src/components/GolfBagSection.astro reusable component (avoids duplication, uses pluralization map for group headers)
  • Update src/pages/golf.astro to import the bag JSON and render the component below Recent Scores in both the live-data and fallback layouts

Before — no My Bag section:

Before

After — My Bag section added below Recent Scores, grouped by club type (Woods / Irons / Wedges / Putters):

After

Original prompt

This section details on the original issue you should resolve

<issue_title>Golf Page - Bag/Clubs section</issue_title>
<issue_description>I'd like to add a section for a mapped golf bag, the information on the bag is:

  • club type (Wood, Iron, Wedge, Putter)
  • club name
  • club model
  • avg distance

The data should come from the appropriate JSON file in the content section that can be updated. The data should be displayed under the rounds play (only if the content exists).

My current bag is mapped:

  • Driver Callaway Rogue ST LS (260)
  • 3 Wood Callaway Rogue (240)
  • 2 iron KZG (220)
  • Irons 4 - GW are Cobra King Tec (200 - 110 gapped evenly)
  • Wedges are 54 and 58 degree KZG
  • Putter Odyssey two ball </issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: kenjdavidson <6210735+kenjdavidson@users.noreply.github.com>
Copilot AI changed the title [WIP] Add mapped golf bag section to golf page feat(golf): add My Bag section sourced from static JSON Mar 16, 2026
Copilot AI requested a review from kenjdavidson March 16, 2026 16:16
@kenjdavidson
kenjdavidson marked this pull request as ready for review March 16, 2026 17:38
@kenjdavidson

Copy link
Copy Markdown
Owner

@copilot did this include UI changes? What did you forget? If you need to update your copilot-instructions so that you actually follow instructions, please do add whatever you need to get this right.

Copilot AI commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

@copilot did this include UI changes? What did you forget? If you need to update your copilot-instructions so that you actually follow instructions, please do add whatever you need...

Sorry about that — I had placeholder URLs in the description instead of real screenshots. The PR description has been updated with actual before/after screenshots:

Before — no My Bag section:

Before

After — My Bag section added below Recent Scores, grouped by club type:

After

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/kenjdavidson.github.io/kenjdavidson.github.io/node_modules/.bin/astro build (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@kenjdavidson
kenjdavidson merged commit 13d56fb into main Mar 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Golf Page - Bag/Clubs section

2 participants