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.

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

Fixing #1733 for non-standard cases #1744

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

Fixing #1733 for non-standard cases #1744

wants to merge 6 commits into from

Conversation

IRRDC
Copy link
Contributor

@IRRDC IRRDC commented Jan 24, 2024

A possible fix for #1733 when the data gets returned in a non-standard way.

Q A
Bug fix? [X]
New feature? [ ]
New sample? [ ]
Related issues? fixes #1733

What's in this Pull Request?

Checking if the data has been returned in a non-standard way and then changing the data structure to conform to the standard way.

A possible fix for pnp#1733 when the data gets returned in a non-standard way.
Added missing references to webURL to service calls.
@michaelmaillot
Copy link
Collaborator

Hi @IRRDC,

Did you have the chance to repro this?

@IRRDC
Copy link
Contributor Author

IRRDC commented Apr 29, 2024

Hi @IRRDC,

Did you have the chance to repro this?

Hi @michaelmaillot,
I'm still getting the results sub-property for some lists when using the DynamicForm within my larger solution using sp-dev-fx-controls-react as an npm package but still not if I'm accessing the list from the controls tester of the sp-dev-fx-controls-react solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants