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 JSON Dot direct array #6476

Merged
merged 2 commits into from
May 22, 2024
Merged

Fix JSON Dot direct array #6476

merged 2 commits into from
May 22, 2024

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented May 21, 2024

Allow consuming a JSON array in which entries are top level
#6474

@Alkarex
Fix JSON Dot direct array
3dcee4a 
Allow consuming a JSON array in which entries are top level
FreshRSS#6474
@Alkarex Alkarex added this to the 1.24.0 milestone May 21, 2024
@Alkarex Alkarex mentioned this pull request May 21, 2024
Closed
@K4LCIFER
Copy link

K4LCIFER commented May 21, 2024 •
edited

Thanks! One note: shoudln't it be $ instead of . to reference the root array? I was looking at this, and it mentions:

$ The root object or array.

@Alkarex
Copy link
Member Author

Alkarex commented May 22, 2024

shoudln't it be $

I have just added support for it as well

@Alkarex Alkarex merged commit c0e87f6 into FreshRSS:edge May 22, 2024
2 checks passed
@Alkarex Alkarex deleted the dot branch May 22, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.24.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Alkarex @K4LCIFER