Skip to content

Commit 6a9dec2

Browse files
authored
Merge pull request #43 from hypercision/roleApprover
Add roleNeededForApprover to shared settings configmap
2 parents 1b1ac63 + 6a607ed commit 6a9dec2

File tree

3 files changed

+7
-1
lines changed

3 files changed

+7
-1
lines changed

charts/thub/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ type: application
66
# This is the chart version. This version number should be incremented each time you make changes
77
# to the chart and its templates, including the app version.
88
# Versions are expected to follow Semantic Versioning (https://semver.org/)
9-
version: 1.7.5
9+
version: 1.7.6
1010

1111
# This is the version number of the application being deployed. This version number should be
1212
# incremented each time you make changes to the application. Versions are not expected to

charts/thub/templates/shared-settings-configmap.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ data:
2525
hclabs.ojt.blockAllTiles: {{ .Values.ojtBlockAllTiles | quote }}
2626
hclabs.ojt.requireTotalHours: {{ .Values.ojtRequireTotalHours | quote }}
2727
hclabs.onlySeeOjtsInYourOrganization: {{ .Values.onlySeeOjtsInYourOrganization | quote }}
28+
{{- if .Values.roleNeededForApprover}}
29+
hclabs.roleNeededForApprover: {{ .Values.roleNeededForApprover | quote }}
30+
{{- end }}
2831
hclabs.sfLearningHistoryConnectorTimeZone: {{ .Values.sfLearningHistoryConnectorTimeZone | quote }}
2932
hclabs.sfLearningHistoryEsUserName: {{ .Values.sfLearningHistoryEsUserName | quote }}
3033
hclabs.sfLearningHistoryLastUpdatedUser: {{ .Values.sfLearningHistoryLastUpdatedUser | quote }}

charts/thub/values.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,9 @@ ojtRequireTotalHours: true
4848
# This includes users who have a `null` `organizationID`; they can only see learners who also do not belong to an organization.
4949
# However, if this is `true` and your user has `ROLE_OJT_FULL_ORG_VISIBILITY`, then you can see all the learners and OJTs.
5050
onlySeeOjtsInYourOrganization: false
51+
# Specify role here to override the default defined by the OJT application.yml
52+
# In order for a Learner/User to be an approver for an OJT or OJTCourse, they need to have this specified role.
53+
roleNeededForApprover: ""
5154
# The timezone that SuccessFactors expects date and time values to be in the learning history connector input file.
5255
# It is in a format that can be used by `java.util.TimeZone.getTimeZone()`. This value can be different for each client.
5356
sfLearningHistoryConnectorTimeZone: "GMT"

0 commit comments

Comments
 (0)