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

Provide a compatibility option for clients with broken data set order. #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ctheune
Copy link

@ctheune ctheune commented Aug 9, 2023

I stumbled over an incorrect form data set construction order in HTMX. I made a bug report there
(bigskysoftware/htmx#1686) but need a work around in the meantime.

They seem to take care of ordering as long as the names are unique.

This is backwards compatible to avoid downstream breakage.

I stumbled over an incorrect form data set construction order
in HTMX. I made a bug report there
(bigskysoftware/htmx#1686) but need a work
around in the meantime.

They seem to take care of ordering as long as the names are unique.

This is backwards compatible to avoid downstream breakage.
@ctheune ctheune changed the title Provide a compatibility options with broken data set order. Provide a compatibility option for clients with broken data set order. Aug 9, 2023
@mcdonc
Copy link
Member

mcdonc commented Aug 10, 2023

Let's see if we can't work this out in the particular consumer of Peppercorn first.

@ctheune
Copy link
Author

ctheune commented Aug 10, 2023

Yeah, absolutely.

tseaver added a commit that referenced this pull request Apr 25, 2024
@ctheune PTAL -- this seems to cause a hang for me in the tests.
@tseaver
Copy link
Member

tseaver commented Apr 25, 2024

@ctheune I've tried to "rebase" your changes atop my recent modernization. The test_bare_without_marker test hangs for me, though. See: 2d246b4

@ctheune
Copy link
Author

ctheune commented Apr 28, 2024

I'll double check this, but it will take a bit. Likely at least until end of May.

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.

None yet

3 participants