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

Recommendations list always empty when queried through Dashboard API #563

Closed
AlvaroBrey opened this issue Oct 21, 2022 · 7 comments
Closed
Labels
0. Needs triage bug Something isn't working

Comments

@AlvaroBrey
Copy link
Member

Both Nextcloud and recommendations at stable25.

Dashboard widget in the UI:

image

Through the Dashboard API:

curl -u test:test http://nextcloud.address/ocs/v2.php/apps/dashboard/api/v1/widget-items\?items\[\]\=recommendations  \
     -H "Accept:application/json"
#
{"ocs":{"meta":{"status":"ok","statuscode":200,"message":"OK"},"data":{"spreed":[]}}}

Not sure what that spreed is doing there, but that is the response I get. Initially found via nextcloud/android#10866

@AlvaroBrey AlvaroBrey added bug Something isn't working 0. Needs triage labels Oct 21, 2022
@AlvaroBrey
Copy link
Member Author

AlvaroBrey commented Oct 21, 2022

Same result with alternative API call:

curl -u test:test http://nextcloud.address/ocs/v2.php/apps/dashboard/api/v1/widget-items \
    -H "Content-Type:application/json" \
    -H "Accept:application/json" \
    -X GET \
    -d '{"sinceIds":{"recommendations":"2022-10-10T15:01:10Z"}}'
#
{"ocs":{"meta":{"status":"ok","statuscode":200,"message":"OK"},"data":{"spreed":[]}}}%    

@ChristophWurst
Copy link
Member

Could this be related to nextcloud/server#33788? Are the file changes older than 2w?

@AlvaroBrey
Copy link
Member Author

Could this be related to nextcloud/server#33788? Are the file changes older than 2w?

No, they are from 2 days ago

@ChristophWurst
Copy link
Member

What are you testing with? Server master or 25? #543 (comment) 🤐

@AlvaroBrey
Copy link
Member Author

What are you testing with? Server master or 25? #543 (comment) zipper_mouth_face

  1. That would explain it 😅 let me test master real quick

@AlvaroBrey
Copy link
Member Author

Can confirm it does work properly on master

@ChristophWurst
Copy link
Member

Fixed with #564

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants