Skip to content

add hall of fame machine detail pages — clickable profiles (50 rtc)#1576

Open
LaphoqueRC wants to merge 3 commits intoScottcjn:mainfrom
LaphoqueRC:fix/issue-505-46af92
Open

add hall of fame machine detail pages — clickable profiles (50 rtc)#1576
LaphoqueRC wants to merge 3 commits intoScottcjn:mainfrom
LaphoqueRC:fix/issue-505-46af92

Conversation

@LaphoqueRC
Copy link
Contributor

Create standalone Flask module with machine detail pages using sqlite3 database access and inline HTML rendering. Implement both web interface and API endpoint following existing Hall of Fame patterns.

resolves #Scottcjn/rustchain-bounties#505

  • hall_of_fame_machine.py
  • hall_of_fame_utils.py
  • tests/test_hall_of_fame_machine.py

testing notes: added tests, ran locally and verified correct behavior. existing tests pass, no regressions found.

rtc wallet: RTC2fe3c33c77666ff76a1cd0999fd4466ee81250ff
sol: HZV6YPdTeJPjPujWjzsFLLKja91K2Ze78XeY8MeFhfK8

additional testing: Tests pass for: machine detail page rendering (200 status), API endpoint JSON response validation, deceased machine memorial styling application, and proper 404 handling for invalid machine IDs. Database queries verified with test data.

ref: Scottcjn/rustchain-bounties#505

@github-actions
Copy link

Welcome to RustChain! Thanks for your first pull request.

Before we review, please make sure:

  • Your PR has a BCOS-L1 or BCOS-L2 label
  • New code files include an SPDX license header
  • You've tested your changes against the live node

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150)

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) tests Test suite changes size/XL PR: 500+ lines labels Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) size/XL PR: 500+ lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant