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

Use actions queue when dispatching undo actions from "Release Actions" #1853

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

whimboo
Copy link
Contributor

@whimboo whimboo commented Oct 30, 2024

Fixes #1837.

@jgraham could you please check? I hope that this is fine - it at least matches what we use in Firefox now when dispatching the actions from the parent process and it works fine. Thanks.


💥 Error: 500 Internal Server Error 💥

PR Preview failed to build. (Last tried on Dec 3, 2024, 3:19 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 Spec Generator - Spec Generator is the web service used to build specs that rely on ReSpec.

🔗 Related URL

Timed out after waiting 30000ms

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

@whimboo whimboo requested a review from jgraham October 30, 2024 09:11
@whimboo
Copy link
Contributor Author

whimboo commented Oct 30, 2024

Well, I actually missed that we also have to enqueue the retrieval of the undo actions.

@whimboo whimboo force-pushed the release-actions-dispatch branch 2 times, most recently from 0c2858c to b1a5a4d Compare October 30, 2024 09:46
@whimboo
Copy link
Contributor Author

whimboo commented Oct 30, 2024

This PR should be ready for review now. Thanks.

index.html Outdated Show resolved Hide resolved
@whimboo
Copy link
Contributor Author

whimboo commented Dec 3, 2024

The current error in the Respec is actually caused by a missing </ol>.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Co-authored-by: jgraham <[email protected]>
index.html Outdated Show resolved Hide resolved
@whimboo whimboo requested a review from jgraham December 3, 2024 15:22
@whimboo
Copy link
Contributor Author

whimboo commented Dec 3, 2024

@jgraham please check again. All suggestions have been applied.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Release Actions" doesn't yet use the queuing mechanism for actions
2 participants