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

Fix issue 1043: ERROR: container must be an array or object #1046

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Fix issue 1043: ERROR: container must be an array or object #1046

merged 1 commit into from
Jul 17, 2023

Conversation

jrgemignani
Copy link
Contributor

@jrgemignani jrgemignani commented Jul 17, 2023 •
edited
Loading

Fixed issue 1043 -

ERROR: container must be an array or object

This was cause by agtype_access_operator not recognizing and decoding a VLE path container.

Added regression tests.

@jrgemignani
Fix issue 1043: ERROR: container must be an array or object
ee82215 
Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
@jrgemignani
Copy link
Contributor Author

If anyone has a few more regression tests for this, that would be great.

@jrgemignani jrgemignani mentioned this pull request Jul 17, 2023
Closed
@jrgemignani
Copy link
Contributor Author

When merging, please make sure not to clear this out -
image

MuhammadTahaNaveed
MuhammadTahaNaveed approved these changes Jul 17, 2023
View reviewed changes
Copy link
Member

@MuhammadTahaNaveed MuhammadTahaNaveed left a comment

Choose a reason for hiding this comment

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

Looks good 👍

rafsun42
rafsun42 approved these changes Jul 17, 2023
View reviewed changes
Copy link
Member

@rafsun42 rafsun42 left a comment

Choose a reason for hiding this comment

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

Look good.

@rafsun42 rafsun42 merged commit 4e26265 into apache:master Jul 17, 2023
M4rcxs pushed a commit to M4rcxs/age that referenced this pull request Jul 18, 2023
@jrgemignani @M4rcxs
Fix issue 1043: ERROR: container must be an array or object (apache#1046
87abc60 
)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
@MuhammadTahaNaveed MuhammadTahaNaveed linked an issue Jul 18, 2023 that may be closed by this pull request
container must be an array or object #1043
Closed
jrgemignani added a commit to jrgemignani/age that referenced this pull request Jul 18, 2023
@jrgemignani
Fix issue 1043: ERROR: container must be an array or object (apache#1046
d2245af 
)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
jrgemignani added a commit to jrgemignani/age that referenced this pull request Jul 18, 2023
@jrgemignani
Fix issue 1043: ERROR: container must be an array or object (apache#1046
281411b 
)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
This was referenced Jul 18, 2023
Merged
Merged
dehowef pushed a commit that referenced this pull request Jul 18, 2023
@jrgemignani
Fix issue 1043: ERROR: container must be an array or object (#1046) (#…
588855a 
…1056)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
dehowef pushed a commit that referenced this pull request Jul 18, 2023
@jrgemignani
Fix issue 1043: ERROR: container must be an array or object (#1046) (#…
62ef391 
…1057)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
muhammadshoaib pushed a commit to muhammadshoaib/age that referenced this pull request Jul 20, 2023
@jrgemignani @muhammadshoaib
Fix issue 1043: ERROR: container must be an array or object (apache#1046
56eba81 
)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

Added regression tests.
MuhammadTahaNaveed pushed a commit to MuhammadTahaNaveed/age that referenced this pull request Aug 24, 2023
@jrgemignani @MuhammadTahaNaveed
Fix issue 1043: ERROR: container must be an array or object (apache#1046
47c9779 
)

Fixed issue 1043 -

ERROR:  container must be an array or object

This was cause by agtype_access_operator not recognizing and
decoding a VLE path container.

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

@MuhammadTahaNaveed MuhammadTahaNaveed MuhammadTahaNaveed approved these changes

@rafsun42 rafsun42 rafsun42 approved these changes

@dehowef dehowef Awaiting requested review from dehowef

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

Successfully merging this pull request may close these issues.

container must be an array or object
3 participants
@jrgemignani @MuhammadTahaNaveed @rafsun42