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

Can't approve a Chart Sync migration in the Chart approval tool #3479

Open
paarriagadap opened this issue Apr 11, 2024 · 4 comments
Open

Can't approve a Chart Sync migration in the Chart approval tool #3479

paarriagadap opened this issue Apr 11, 2024 · 4 comments
Labels
admin Issues that need to be solved in the grapher admin (mostly) bug

Comments

@paarriagadap
Copy link

Description

After using Chart Sync, I can't approve a series of charts in the chart approval tool.

Expected behaviour

Charts approved after pressing the green Approve button

Steps to reproduce

Steps to reproduce the behavior:

  1. In the Chart Sync tool, select Source: update-oecd-idd and Target: live and press Sync charts
  2. Go to the chart approval tool
  3. Select "Approve"
  4. You will see this error:
Failed to POST /api/suggested-chart-revisions/36619/update (404)
Error: Suggest chart revision 36619 cannot be updated with status="approved".
    at  (/home/owid/owid-grapher/adminSiteServer/apiRouter.ts:916:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  1. The same happens with Reject:
Failed to POST /api/suggested-chart-revisions/36619/update (404)
Error: Suggest chart revision 36619 cannot be updated with status="rejected".
    at  (/home/owid/owid-grapher/adminSiteServer/apiRouter.ts:916:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Screenshots

Screen.Recording.2024-04-11.at.10.55.41.mov
Screen.Recording.2024-04-11.at.10.55.10.mov
Screen.Recording.2024-04-11.at.10.59.04.mov

Environment

For desktop, provide:

  • OS: macOS Sonoma 14.4.1
  • Browser Chrome
  • Version 123.0.6312.122 (Official Build) (arm64)

Additional context

I can't remove these suggestions in the tool, because they can't be approved, rejected or flagged

@paarriagadap
Copy link
Author

paarriagadap commented Apr 11, 2024

Update: After some ~30 minutes I can see the charts approved and the list in the tool empty. I don't know if there still something to check, but it was weird behaviour.

@paarriagadap
Copy link
Author

This issue is probably related to a caching issue in the new https://admin.owid.io, because it took 30 minutes to be solved. This is discussed here in Slack.

@danyx23 danyx23 added the admin Issues that need to be solved in the grapher admin (mostly) label May 7, 2024
@danyx23
Copy link
Contributor

danyx23 commented May 7, 2024

@lucasrodes and @Marigold can you triage this issue and tell me if this is still relevant? Thanks!

@Marigold
Copy link
Contributor

Marigold commented May 8, 2024

I'd wait a bit more as we're trying to deprecate chart approval tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin Issues that need to be solved in the grapher admin (mostly) bug
Projects
None yet
Development

No branches or pull requests

3 participants