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

[BUG] - Mealie is repeatedly calling /api/groups/self on home page when scrolling #3317

Closed
5 of 6 tasks
cmintey opened this issue Mar 15, 2024 · 1 comment · Fixed by #3321
Closed
5 of 6 tasks

[BUG] - Mealie is repeatedly calling /api/groups/self on home page when scrolling #3317

cmintey opened this issue Mar 15, 2024 · 1 comment · Fixed by #3321
Labels
bug: confirmed bug Something isn't working

Comments

@cmintey
Copy link
Contributor

cmintey commented Mar 15, 2024

First Check

  • This is not a feature request.
  • I added a very descriptive title to this issue (title field is above this).
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the Mealie documentation, with the integrated search.
  • I already read the docs and didn't find an answer.
  • This issue can be replicated on the demo site (https://demo.mealie.io/).

What is the issue you are experiencing?

On the latest nightly build, scrolling through recipes is making dozens of calls to /api/groups/self. I believe this is related to the changes in #3303.
I fired up a local dev instance and looking at the stack trace in the developer console on Firefox shows the calls emitting from RecipeContextMenu.vue. This call should probably be cached and re-used.

Steps to Reproduce

  1. Log in to Mealie on latest nightly build
  2. Open developer console and go to Network tab
  3. Scroll through your recipes
  4. Observe the dozens of calls to /api/groups/self. One for each recipe card that gets rendered

Please provide relevant logs

mealie     | INFO:     10.0.2.1:0 - "GET /api/recipes?page=1&perPage=64&orderBy=created_at&orderDirection=desc&paginationSeed=1710473439668&searchSeed=1710473439668&search=&requireAllCategories=false&requireAllTags=false&requireAllTools=false&requireAllFoods=false HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
mealie     | INFO:     10.0.2.1:0 - "GET /api/groups/self HTTP/1.1" 200 OK
...

Mealie Version

Nightly
Build: 1af0f42

Deployment

Docker (Linux)

Additional Deployment Details

No response

@cmintey cmintey added bug Something isn't working triage labels Mar 15, 2024
@michael-genson
Copy link
Collaborator

Maybe the group is insecure and has to look at itself constantly to feel validated

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

Successfully merging a pull request may close this issue.

2 participants