Skip to content

Conversation

Learath2
Copy link

@Learath2 Learath2 commented May 7, 2024

This feature allows selecting structs without breaking them up. It's useful for types like time.Time, where having it as a selectColumn generates nothing in the marshalled select clause.

This feature allows selecting structs without breaking them up
Copy link

salesforce-cla bot commented May 7, 2024

Thanks for the contribution! Before we can merge this, we need @Learath2 to sign the Salesforce Inc. Contributor License Agreement.

Copy link
Collaborator

@atulkc atulkc left a comment

Choose a reason for hiding this comment

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

Please sign the CLA as required to contribute to this repo.

})
})

Context("when nested struct with soql fields is selected opaquely", func() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you add a test that verifies the entire SOQL generation? Basically add a Context to Describe("Marshal")

Copy link
Author

Choose a reason for hiding this comment

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

I will look into it 👍

@Learath2
Copy link
Author

Learath2 commented May 7, 2024

I did sign the CLA and if I click the link again it says you have already signed the CLA. It seems broken somehow

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.

2 participants