Skip to content

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

Open
ctheune wants to merge 1 commit into
Pylons:masterfrom
ctheune:htmx-workaround-1686-wrong-form-dataset-construction
Open

Provide a compatibility option for clients with broken data set order.#20
ctheune wants to merge 1 commit into
Pylons:masterfrom
ctheune:htmx-workaround-1686-wrong-form-dataset-construction

Conversation

@ctheune

@ctheune ctheune commented Aug 9, 2023

Copy link
Copy Markdown

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

mcdonc commented Aug 10, 2023

Copy link
Copy Markdown
Member

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

@ctheune

ctheune commented Aug 10, 2023

Copy link
Copy Markdown
Author

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

tseaver commented Apr 25, 2024

Copy link
Copy Markdown
Member

@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

ctheune commented Apr 28, 2024

Copy link
Copy Markdown
Author

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.

3 participants