Skip to content

Conversation

@acoumb
Copy link
Contributor

@acoumb acoumb commented Nov 7, 2024

Description

Updated Umbraco UIBuilder docs about related collections.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Deadline (if relevant)

@sofietoft
Copy link
Contributor

Looks good @acoumb ! 🙌

Should we add this to version 14 and 15 as well?

@acoumb
Copy link
Contributor Author

acoumb commented Nov 8, 2024

Yes, please.

@sofietoft
Copy link
Contributor

Perfect. I'll take care of that after it's been merged.

Thanks again!

@sofietoft sofietoft merged commit 551992b into umbraco:main Nov 8, 2024
3 checks passed
@jemayn
Copy link
Contributor

jemayn commented Nov 8, 2024

Hey @acoumb

Thanks for writing this documentation!
Any chance there could be an explanation of what the string relationAlias param does in the GetRelationsByParentIdImpl method?

I see it is not currently in use in your example 🙂

@acoumb
Copy link
Contributor Author

acoumb commented Nov 8, 2024

Hi,

It's the relation alias you define here:

    relationConfig
        .SetAlias("studentsCourses")
        .SetJunction<StudentCourse>(x => x.StudentId, y => y.CourseId);

sofietoft added a commit that referenced this pull request Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants