Skip to content

Fix dot submissions: restore auth, add public feedback, auto-populate from inbox - #9

Merged
devyanijain merged 4 commits into
devyanijain-build-portfolio-sitefrom
devyanijain-fix-dot-submission-persistence
Aug 31, 2026
Merged

Fix dot submissions: restore auth, add public feedback, auto-populate from inbox#9
devyanijain merged 4 commits into
devyanijain-build-portfolio-sitefrom
devyanijain-fix-dot-submission-persistence

Conversation

@devyanijain

Copy link
Copy Markdown
Owner

What broke

GitHub secret scanning replaced the Authorization header value with ****** but didn't close the backtick template literal — causing a JS syntax error that silently made every submission fail to post.

What's fixed

🔧 Bug fixes

  • Restored broken Authorization header in postPrivateFeedback — submissions now actually post to the inbox
  • Added public feedback text (pub.t) to the GitHub issue body so the "what was it like" field is captured going forward

✨ New: auto-populate submitted dots for all visitors

Instead of dots only appearing for the person who submitted them (in-memory only), the page now fetches golden-seeds-inbox issues on every page load and hydrates the dot field dynamically:

  • Dots render immediately (blank), then light up after the async fetch
  • Each issue is mapped by relationship → color → assigned to the next available dot of that color in order
  • Rachel Florence, Ricky Zhang, and April's dots appear for every visitor — and any future submissions show up automatically on next page load, no code change needed
  • Anonymous and [Test] issues are skipped
  • All failures are silent — the page never breaks if the fetch fails

Previously submitted dots

All 3 real submissions (Rachel Florence, Ricky Zhang, April) are preserved — they live in the GitHub Issues inbox and will be fetched and shown dynamically.

Devyani Jainand others added 4 commits August 31, 2026 09:21
GitHub secret scanning replaced the token value with '******' but did
not close the backtick template literal, causing a JS syntax error that
silently swallowed the entire headers object and made all feedback
submissions fail.
Restore the correct header: `token ${_tk}`
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
The public 'what was it like working with Devyani' field (pub.t) was
never written to the inbox issue. Add a '💬 Public feedback' section
so all feedback is captured in one place.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e
Mark the first teal/red/purple dots as submitted with data from the
golden-seeds-inbox issues so they appear as planted on page load.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e
Instead of hardcoding submitted dots, fetch the golden-seeds-inbox
issues on every page load and hydrate the dot field in-place. This
means any new submission automatically appears for all visitors
without needing a code change.
- submittedDots starts empty; loadSubmittedDots() fills it async
- Dots render immediately (blank), then light up after the fetch
- Issues are assigned to dots in creation order, per color/relationship
- Anonymous and [Test] issues are skipped
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e
@devyanijain
devyanijain merged commit 946a403 into devyanijain-build-portfolio-siteAug 31, 2026
Sign up for freeto 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.

2 participants

@devyanijain@devyanixjain