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 temporary fix for Agave issue 479 #3184

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

lorisleiva
Copy link
Contributor

@lorisleiva lorisleiva commented Sep 1, 2024

This PR removes the temporary request transformer that was waiting for this PR to be merged and available on mainnet-beta.

Copy link

changeset-bot bot commented Sep 1, 2024

🦋 Changeset detected

Latest commit: 39284be

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@mcintyre94 mcintyre94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@lorisleiva lorisleiva force-pushed the loris/remove-temp-fix-479 branch 2 times, most recently from f3051ba to 38241cb Compare September 3, 2024 22:11
@lorisleiva lorisleiva changed the base branch from master to loris/update-get-cluster-nodes September 3, 2024 22:11
@lorisleiva lorisleiva changed the base branch from loris/update-get-cluster-nodes to graphite-base/3184 September 5, 2024 09:19
@lorisleiva lorisleiva force-pushed the loris/remove-temp-fix-479 branch from 38241cb to 54addbf Compare September 5, 2024 09:19
@lorisleiva lorisleiva changed the base branch from graphite-base/3184 to master September 5, 2024 09:20
@lorisleiva lorisleiva force-pushed the loris/remove-temp-fix-479 branch 2 times, most recently from e36ee23 to bf4897c Compare September 6, 2024 11:48
Copy link
Collaborator

@steveluscher steveluscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added via Giphy

Copy link
Contributor Author

lorisleiva commented Sep 10, 2024

Merge activity

  • Sep 10, 3:48 AM EDT: @lorisleiva started a stack merge that includes this pull request via Graphite.
  • Sep 10, 3:48 AM EDT: Graphite rebased this pull request as part of a merge.
  • Sep 10, 3:49 AM EDT: @lorisleiva merged this pull request with Graphite.

@lorisleiva lorisleiva force-pushed the loris/remove-temp-fix-479 branch from bf4897c to 39284be Compare September 10, 2024 07:48
@lorisleiva lorisleiva merged commit 28ca5d1 into master Sep 10, 2024
7 checks passed
@lorisleiva lorisleiva deleted the loris/remove-temp-fix-479 branch September 10, 2024 07:49
@github-actions github-actions bot mentioned this pull request Sep 10, 2024
Copy link
Contributor

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants