Skip to content
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

Jump to bottom

Resolve conflicts between logical and display names in connectors #2362

Closed
wants to merge 7 commits into from
Closed

Resolve conflicts between logical and display names in connectors #2362

wants to merge 7 commits into from

Conversation

LucGenetier
Copy link
Contributor

@LucGenetier LucGenetier commented May 2, 2024 •
edited
Loading

"ServiceDeskPlus Cloud" connector generate failures with a combination of props like this

  • logical name = "held_by", display name = "technician"
  • logical name = "technician", display name = "technician"

@LucGenetier LucGenetier requested a review from a team as a code owner May 2, 2024 12:38
@LucGenetier
Copy link
Contributor Author

✅ No public API change.

@MikeStall
Copy link
Contributor

Can you add a PR description?
What policy are you using to resolve? What does Power Apps do?

@LucGenetier
Copy link
Contributor Author

LucGenetier commented May 3, 2024 •
edited
Loading

Done.
Logical names have priority and won't be renamed. Only display names could.
Power Apps doesn't rename display names, they have conflicts but can't use them in expressions

In reply to: 2090624811

@LucGenetier
Copy link
Contributor Author

✅ No public API change.

@LucGenetier
Copy link
Contributor Author

✅ No public API change.

@LucGenetier
Copy link
Contributor Author

✅ No public API change.

@LucGenetier
Copy link
Contributor Author

Changes already merged, closing PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lesaltzm lesaltzm lesaltzm left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LucGenetier @MikeStall @lesaltzm