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

Remove cache form Reports log form #190

Closed
DiegoPino opened this issue Jan 19, 2024 · 1 comment
Closed

Remove cache form Reports log form #190

DiegoPino opened this issue Jan 19, 2024 · 1 comment
Assignees
Labels
AMI entity bug Something isn't working Reporting Errors, Logs, etc. UI Buttons and pixels
Milestone

Comments

@DiegoPino
Copy link
Member

What?

Because the Reports log tab on an AMI set is an Entity Form (means, it depends on the Entity) there is an edge case where someone visits it before any processing has happened (means the Log is still not created) and Drupal caches (because it uses the entity and the user context) the empty one. Another user, visiting it afterwards will see the Reports, but the original user will keep seeing an empty Form until global caches are cleared (or user caches, tags) or the entity itself changes (someone changes the status, updates a CSV or the JSON config)

The solution so far (but needs more testing) is very simple, make cache last 0.

@DiegoPino DiegoPino added this to the 0.7.0 milestone Jan 19, 2024
@DiegoPino DiegoPino self-assigned this Jan 19, 2024
@DiegoPino DiegoPino added bug Something isn't working Reporting Errors, Logs, etc. AMI entity UI Buttons and pixels labels Jan 19, 2024
@DiegoPino
Copy link
Member Author

Resolved (so far) via 830e6d9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AMI entity bug Something isn't working Reporting Errors, Logs, etc. UI Buttons and pixels
Projects
None yet
Development

No branches or pull requests

1 participant