Skip to content

Fix broken references to file upload variants#15

Merged
MartinSStewart merged 1 commit intolamdera:mainfrom
dillonkearns:patch-1
Nov 3, 2025
Merged

Fix broken references to file upload variants#15
MartinSStewart merged 1 commit intolamdera:mainfrom
dillonkearns:patch-1

Conversation

@dillonkearns
Copy link
Copy Markdown
Contributor

When I ran the instructions in the repo and example folder (running lamdera live from examples/counter and going to http://localhost:8000/tests/EndToEndTests.elm), I got these errors:

-- NAMING ERROR  program-test/example/counter/tests/EndToEndTests.elm

I cannot find a `CancelFileUpload` variant:

35|     , handleFileUpload = always CancelFileUpload
                                    ^^^^^^^^^^^^^^^^
These names seem close though:

    UnhandledFileUpload
    UnhandledMultiFileUpload
    Dom.Viewport
    BadUrlResponse

Note: Evergreen migrations need access to all custom type variants. Make sure
both `CancelFileUpload` and `Evergreen.VX.CancelFileUpload` are exposed.

Hint: Read <https://elm-lang.org/0.19.1/imports> to see how `import`
declarations work in Elm.


-- NAMING ERROR  program-test/example/counter/tests/EndToEndTests.elm

I cannot find a `CancelMultipleFilesUpload` variant:

36|     , handleMultipleFilesUpload = always CancelMultipleFilesUpload
                                             ^^^^^^^^^^^^^^^^^^^^^^^^^
These names seem close though:

    UnhandledMultiFileUpload
    UploadMultipleFiles
    UnhandledFileUpload
    Effect.Test.UnhandledFileUpload

Note: Evergreen migrations need access to all custom type variants. Make sure
both `CancelMultipleFilesUpload` and `Evergreen.VX.CancelMultipleFilesUpload`
are exposed.

Hint: Read <https://elm-lang.org/0.19.1/imports> to see how `import`
declarations work in Elm.

Based on the types required here these seem to be the new variant names that are needed.

@MartinSStewart MartinSStewart merged commit b39971f into lamdera:main Nov 3, 2025
@MartinSStewart
Copy link
Copy Markdown
Member

Thanks! I really should get around to adding CI that checks the examples so they don't get out of sync.

@dillonkearns dillonkearns mentioned this pull request Nov 3, 2025
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.

2 participants