Skip to content

Comments

YesImportQualifiedPost in /integration#3793

Closed
fisx wants to merge 1 commit intodevelopfrom
YesImportQualifiedPost
Closed

YesImportQualifiedPost in /integration#3793
fisx wants to merge 1 commit intodevelopfrom
YesImportQualifiedPost

Conversation

@fisx
Copy link
Contributor

@fisx fisx commented Jan 5, 2024

https://wearezeta.atlassian.net/browse/WPB-5803

We should either support this extension in both /integration and everywhere else, or nowhere. I'm very much in favor of "everywhere". Note that the extension allows for the old and the new syntax.

(I'm also curious: why was it disabled?)

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 5, 2024
@fisx fisx requested review from pcapriotti and smatting January 5, 2024 16:16
Copy link
Contributor

@smatting smatting left a comment

Choose a reason for hiding this comment

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

@fisx
Copy link
Contributor Author

fisx commented Jan 5, 2024

ah, ok. makes kinda sense...

sorry for the noise.

@fisx fisx closed this Jan 5, 2024
@echoes-hq echoes-hq bot added the echoes: unplanned Any work item that isn’t part of the product or technical roadmap. label Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: unplanned Any work item that isn’t part of the product or technical roadmap. ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants