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

[MRXN23-228] fix: project export loop on status actions #1427

Merged

Conversation

andresgnlez
Copy link
Member

@andresgnlez andresgnlez commented Aug 7, 2023

Substitute this line for a meaningful title for your changes

Overview

This PRs fixes a loop with one of the dependencies when the user generates the download of a project and the job callbacks are triggered in a loop.

Designs

Testing instructions

Please explain how to test the PR: ID of a dataset, steps to reach the feature, etc.

Feature relevant tickets

MRXN23-291


Checklist before submitting

  • Meaningful commits and code rebased on develop.
  • If this PR adds feature that should be tested for regressions when
    deploying to staging/production, please add brief testing instructions
    to the deploy checklist (docs/deployment-checklist.md)
  • Update CHANGELOG file

@andresgnlez andresgnlez added the Frontend Everything related frontend label Aug 7, 2023
@andresgnlez andresgnlez self-assigned this Aug 7, 2023
@vercel
Copy link

vercel bot commented Aug 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
marxan ✅ Ready (Inspect) Visit Preview Aug 7, 2023 11:06am

@andresgnlez andresgnlez marked this pull request as ready for review August 7, 2023 11:06
@andresgnlez andresgnlez merged commit e2f1cdd into MRXN23-228-app-new-layout-for Aug 7, 2023
9 checks passed
@andresgnlez andresgnlez deleted the MRXN23-291-export-project-loop-v2 branch August 7, 2023 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Everything related frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant