Skip to content

Deleted custom code location#4

Open
sjoerdvandijkt4e wants to merge 1 commit into
mainfrom
patch-deleted-custom-code
Open

Deleted custom code location#4
sjoerdvandijkt4e wants to merge 1 commit into
mainfrom
patch-deleted-custom-code

Conversation

@sjoerdvandijkt4e

Copy link
Copy Markdown
  • Location information and custom_fields deleted from code (this was custom made)
  • Basic connector is now usable without extra adjustments to the code

@rhouthuijzen rhouthuijzen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zou je naar de feedback kunnen kijken. Ook nog even de README nalopen m.b.t. custom fields. Mogelijk de tekst iets finetunen met het is mogelijk om een custom veld te gebruiken zie voorbeeld in de code

Comment thread update.ps1
try {
$splatGetCostcentres = @{
Url = "$($actionContext.Configuration.BaseUrl)/api2/costcentres?name=$($actionContext.Data.mapping.costCenterId)"
Url = "$($actionContext.Configuration.BaseUrl)/api2/costcentres?name=$($actionContext.Data.mapping.costCenterId)&name=$($actionContext.Data.mapping.costCenterId)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Als ik de code goed leest doe je nu twee keer exact dezelfde name meegeven

name=$($actionContext.Data.mapping.costCenterId)&
name=$($actionContext.Data.mapping.costCenterId)

Denk dat deze wijziging ongedaan gemaakt kan worden

Comment thread update.ps1
try {
$splatGetDepartments = @{
Url = "$($actionContext.Configuration.BaseUrl)/api2/departments?name=$($actionContext.Data.mapping.departmentId)"
Url = "$($actionContext.Configuration.BaseUrl)/api2/departments?name=$($actionContext.Data.mapping.departmentId)&name=$($actionContext.Data.mapping.departmentId)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Als ik de code goed leest doe je nu twee keer exact dezelfde name meegeven

name=$($actionContext.Data.mapping.departmentId)&
name=$($actionContext.Data.mapping.departmentId)

Denk dat deze wijziging ongedaan gemaakt kan worden

Comment thread update.ps1
# else {
# Write-Information "LocationId (name) is empty lookup skiped"
# }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik zou het voorbeeld van custom_fields er ook inlaten. Desnoods met een extra note. Want nu haalt je ui gecommentarieerde code alleen de locatie op en wordt het niet aan action context data toegevoegd

Comment thread create.ps1
# PowerShell V2
#################################################
#
# Version: 1.2.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Terugdraaien. # PowerShell V2 zonder versie is de template

Comment thread disable.ps1
# HelloID-Conn-Prov-Target-Facilitor-Disable
# PowerShell V2
#
# Version: 1.2.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Terugdraaien. # PowerShell V2 zonder versie is de template

Comment thread delete.ps1
# HelloID-Conn-Prov-Target-Facilitor-Delete
# PowerShell V2
#
# Version: 1.2.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Terugdraaien. # PowerShell V2 zonder versie is de template

Comment thread create.ps1
try {
$splatGetCostcentres = @{
Url = "$($actionContext.Configuration.BaseUrl)/api2/costcentres?name=$($actionContext.Data.mapping.costCenterId)"
Url = "$($actionContext.Configuration.BaseUrl)/api2/costcentres?name=$($actionContext.Data.mapping.costCenterId)&name=$($actionContext.Data.mapping.costCenterId)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zie opm update script

Comment thread create.ps1
try {
$splatGetDepartments = @{
Url = "$($actionContext.Configuration.BaseUrl)/api2/departments?name=$($actionContext.Data.mapping.departmentId)"
Url = "$($actionContext.Configuration.BaseUrl)/api2/departments?name=$($actionContext.Data.mapping.departmentId)&name=$($actionContext.Data.mapping.departmentId)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zie opm update script

Comment thread create.ps1
# SearchProperty = 'name'
# }
# $location = Get-FacilitorResource @splatGetLocations

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zie opm update script

Comment thread create.ps1
Write-Information 'Creating and correlating Facilitor account'
$splatCreateUser = @{
Uri = "$($actionContext.Configuration.BaseUrl)/api2/persons?inlcude=custom_fields"
Uri = "$($actionContext.Configuration.BaseUrl)/api2/persons?include=custom_fields"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Include=custom_fields maakt dit nog uit dat we dit meegeven? Als het geen problemen gaf gewoon laten staan

@rhouthuijzen rhouthuijzen added the pending-author-update PR is waiting for the author to make necessary modifications or updates label Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-author-update PR is waiting for the author to make necessary modifications or updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants