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

Merged
saurabhraghuvanshii merged 5 commits into from
Jun 24, 2026
+12 −100
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 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
c1118ad
Merge branch 'master' into fix/permissions-csv-shortcodes
Rajesh-Nagarajan-11 Jun 24, 2026
daddd36
Merge branch 'master' into fix/permissions-csv-shortcodes
saurabhraghuvanshii Jun 24, 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 hasPrefix $sectionName "academy" }}
{{ $urlPath = "roles/academy-roles" }}
{{ else if or (hasPrefix $sectionName "learner") (hasPrefix $sectionName "leaner") }}
{{ $urlPath = "roles/academy-roles/#learner" }}
{{ 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 hasPrefix $sectionName "academy" }}
{{ $urlPath = "roles/academy-roles" }}
{{ else if or (hasPrefix $sectionName "learner") (hasPrefix $sectionName "leaner") }}
{{ $urlPath = "roles/academy-roles/#learner" }}
{{ 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