Skip to content

Fix permissions table to use workflow-generated keys.csv #1111

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.

Sign up for GitHub

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

Open
rishiraj38 wants to merge 3 commits into
base: master
Choose a base branch
from
+12 −100
Open

Fix permissions table to use workflow-generated keys.csv #1111

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
eebf3b0
Use auto-generated keys.csv in permissions shortcodes
rishiraj38 Jun 22, 2026
1bd9844
Apply suggestion from @gemini-code-assist[bot]
rishiraj38 Jun 22, 2026
4b0ddce
Fix redundant or operator and handle learner typo in prefix check
rishiraj38 Jun 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions layouts/shortcodes/csvtable-roles.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{ $data := "" }}
{{ $p := "static/data/csv/keys-backup.csv" }}
{{ $excludedColumns := slice 0 10 11 12 13 14 15 16 17 18 }}
{{ $p := "static/data/csv/keys.csv" }}
{{ $excludedColumns := slice 0 11 12 13 14 15 16 17 18 19 20 }}

{{ if os.FileExists $p }}
{{ $opts := dict "delimiter" "," }}
Expand Down Expand Up @@ -39,6 +39,10 @@
{{ $urlPath = "roles/organization-roles" }}
{{ else if hasPrefix $sectionName "provider" }}
{{ $urlPath = "roles/#provider-admin-role" }}
{{ else if or (hasPrefix $sectionName "academy") }}
{{ $urlPath = "roles/academy-roles" }}
{{ else if or (hasPrefix $sectionName "leaner") }}
{{ $urlPath = "roles/academy-roles/#learner" }}
Comment thread
rishiraj38 marked this conversation as resolved.
Outdated
{{ else }}
{{ $urlPath = print "roles/" $sectionName | urlize }}
{{ end }}
Expand Down
8 changes: 6 additions & 2 deletions layouts/shortcodes/csvtable.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{ $data := "" }}

{{ $p := "static/data/csv/keys-backup.csv" }}
{{ $p := "static/data/csv/keys.csv" }}
{{ $excludedColumns := slice "Category" "Keychain ID" "Key ID" "Inserted" "Local Provider" }} <!-- Add the names of the columns to exclude -->

{{ if os.FileExists $p }}
Expand Down Expand Up @@ -52,6 +52,10 @@ <h2>{{ $category }} Permissions</h2>
{{ $urlPath = "roles/#provider-admin-role" }}
{{ else if hasPrefix $sectionName "user" }}
{{ $urlPath = "roles/user-role" }}
{{ else if or (hasPrefix $sectionName "academy") }}
{{ $urlPath = "roles/academy-roles" }}
{{ else if or (hasPrefix $sectionName "leaner") }}
{{ $urlPath = "roles/academy-roles/#learner" }}
Comment thread
rishiraj38 marked this conversation as resolved.
Outdated
{{ else }}
{{ $urlPath = print "roles/" }}
{{ end }}
Expand All @@ -69,7 +73,7 @@ <h2>{{ $category }} Permissions</h2>
{{ $col := index $header $j }}
{{ if not (in $excludedColumns $col) }}
<td>
{{ if and (gt $j 2) (lt $j 10) }}
{{ if and (gt $j 2) (lt $j 11) }}
{{ if eq $cell "X" }}
✅
{{ else if eq $cell "X*" }}
Expand Down
96 changes: 0 additions & 96 deletions static/data/csv/keys-backup.csv
View file
Open in desktop

This file was deleted.

Loading