-
Notifications
You must be signed in to change notification settings - Fork 191
Modern HIPs site built with Vite #1423
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 52 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
da992a7
Add Vite project scaffold and HIP data build script
mgarbs 985f179
Add site HTML with pipeline, filters, and detail view
mgarbs 54bfdfe
Add CSS theme with dark mode and responsive layout
mgarbs 28bd7f9
Add application logic with GitHub integration
mgarbs 5252c93
Add logo and favicon assets with dark mode variant
mgarbs 47ebdde
Add GitHub Actions workflow for Vite site deployment
mgarbs 97b3ceb
Remove Jekyll configuration, layouts, and templates
mgarbs c2145fa
Map legacy needs-council-approval to needs-hedera-review
mgarbs 30305af
Format author and working group with linked GitHub usernames
mgarbs f4fc244
Show only name for authors, linked to GitHub profile or email
mgarbs bddb859
Make release version clickable to correct repo release notes
mgarbs 8627d56
Handle parenthesized GitHub handles in author formatting
mgarbs ca8f921
Add threaded discussion and review comments with snake flow layout
mgarbs 3380229
Polish comment and thread styling for full readability
mgarbs d19fea1
Fix comment text cutoff in review threads
mgarbs 98a66b9
Render GitHub suggestion blocks as styled diffs
mgarbs 5c43675
Strip GitHub email notification footers from comments
mgarbs 911e313
Fix email footer stripping for blockquoted replies
mgarbs 292fb8a
Support configurable base path for subdirectory deployments
mgarbs 61c8746
Remove subtitle text under logo
mgarbs 0a55d20
Fix 404 on edit actions for draft HIPs
mgarbs ca6a383
Fix draft HIP edit links to point to PR file changes
mgarbs 1f39843
Store PR number for draft HIPs to fix edit links
mgarbs 2bb72a9
Update Last Call tooltip text
mgarbs 756933d
Add syntax highlighting with highlight.js and vibrant theme
mgarbs aa0ae4d
Add VS Code-style code enhancements for code blocks
mgarbs d10a088
Add HIP Creation Wizard with guided form and Discord sharing
mgarbs 94d7502
Clarify approval/review labels to avoid ambiguity
mgarbs e817871
Fix About page logo for dark mode
mgarbs 4f3684e
Add HIPs text next to Hiero logo in header
mgarbs 9effef7
Refine header logo: crop whitespace, align HIPs text with Hiero
mgarbs b6cb40f
Update netlify.toml to build Vite site instead of Jekyll
mgarbs 01a31b2
Merge branch 'main' into feat/modern-hips-site
mgarbs d8b8ef3
Update netlify.toml to build Vite site instead of Jekyll
mgarbs c793f9e
Add HIPs text to logo image and clean up header alignment
mgarbs 9246534
Fix CI checks: skip deleted files in validation, suppress Codacy warn…
mgarbs 0f03146
Merge branch 'main' into feat/modern-hips-site
mgarbs 73f4353
Fix XSS vulnerabilities: sanitize all innerHTML with DOMPurify
mgarbs d3636d0
Fix author links for bare GitHub usernames in angle brackets
mgarbs 9a25b10
Pass GITHUB_TOKEN to build:data for discussion comment fetching
mgarbs c560cfa
Redirect old repo discussion URLs and improve error logging
mgarbs 1086e80
Reduce GraphQL query cost and fix rate limit break logic
mgarbs e395d44
Fix broken images in HIPs by copying assets and rewriting paths
mgarbs 59f4f59
Replace broken HIP state diagram images with ASCII art
mgarbs e7d422f
Add Mermaid state diagrams with tooltips and zoom for HIP-1
mgarbs 9f60fd2
Rebuild logo images with unified font for Hiero and HIPs
mgarbs 0daff33
Enable search from individual HIP detail pages
mgarbs 7a0b4f4
Return to original HIP when search is cleared from detail view
mgarbs 1b0e43b
Fix Codacy static analysis warnings
mgarbs e452056
Replace innerHTML with DOMParser to fix Codacy XSS warning
mgarbs 8ac11c2
Fix table rendering with DOMParser by wrapping table elements
mgarbs ed8b498
Fix Codacy static analysis warnings without breaking site
mgarbs 678e779
Update .github/workflows/deploy-site.yml
mgarbs 2c3fcba
Update .github/workflows/deploy-site.yml
mgarbs 61cefa3
Update .github/workflows/deploy-site.yml
mgarbs 0031c23
Update .github/workflows/deploy-site.yml
mgarbs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| exclude_paths: | ||
| - "site/package-lock.json" | ||
| - "site/node_modules/**" | ||
| - "site/dist/**" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,65 @@ | ||
| name: Deploy Site | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main] | ||
| workflow_dispatch: | ||
|
|
||
| permissions: | ||
| contents: read | ||
| pages: write | ||
| id-token: write | ||
|
|
||
| concurrency: | ||
| group: "pages" | ||
| cancel-in-progress: false | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
|
mgarbs marked this conversation as resolved.
Outdated
|
||
| steps: | ||
| - name: Harden the runner (Audit all outbound calls) | ||
| uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2 | ||
| with: | ||
| egress-policy: audit | ||
|
|
||
| - name: Checkout | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 | ||
| with: | ||
| node-version: "20" | ||
|
|
||
| - name: Install dependencies | ||
| working-directory: site | ||
| run: npm ci | ||
|
|
||
| - name: Build data | ||
| working-directory: site | ||
| run: npm run build:data | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Build site | ||
| working-directory: site | ||
| run: npm run build | ||
|
|
||
| - name: Setup Pages | ||
| uses: actions/configure-pages@983d7736d9b0ae728b81ab479565c72886d7745b # v5.0.0 | ||
|
|
||
| - name: Upload artifact | ||
| uses: actions/upload-pages-artifact@56afc609e74202658d3ffba0e8f6dda462b719fa # v3.0.1 | ||
| with: | ||
| path: site/dist | ||
|
|
||
| deploy: | ||
| environment: | ||
| name: github-pages | ||
| url: ${{ steps.deployment.outputs.page_url }} | ||
| runs-on: ubuntu-latest | ||
|
mgarbs marked this conversation as resolved.
Outdated
|
||
| needs: build | ||
| steps: | ||
| - name: Deploy to GitHub Pages | ||
| id: deployment | ||
| uses: actions/deploy-pages@d6db90164ac5ed86f2b6aed7e0febac2b3c603fc # v4.0.5 | ||
|
mgarbs marked this conversation as resolved.
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.