Skip to content

add devfestnantes 2025#1638

Merged
martinbonnin merged 1 commit intojoreilly:mainfrom
ArtyMaury:add-devfestnantes-2025
Jul 25, 2025
Merged

add devfestnantes 2025#1638
martinbonnin merged 1 commit intojoreilly:mainfrom
ArtyMaury:add-devfestnantes-2025

Conversation

@ArtyMaury
Copy link
Copy Markdown
Contributor

No description provided.

@ArtyMaury
Copy link
Copy Markdown
Contributor Author

@RobinCaroff I'm not sure how I should test if I missed anything, but should be alright

@martinbonnin
Copy link
Copy Markdown
Collaborator

Merging this quickly, we'll see how that worked

@martinbonnin martinbonnin merged commit bb32d34 into joreilly:main Jul 25, 2025
4 of 5 checks passed
@martinbonnin
Copy link
Copy Markdown
Collaborator

@ArtyMaury importing the conference data fails on

java.lang.IllegalStateException: Cannot get https://raw.githubusercontent.com/GDG-Nantes/Devfest2025/main/data/slots.json: 404: Not Found
        at dev.johnoreilly.confetti.backend.import.DevFestNantes.getUrl(DevFestNantes.kt:130)
        at dev.johnoreilly.confetti.backend.import.DevFestNantes.access$getUrl(DevFestNantes.kt:115)
        at dev.johnoreilly.confetti.backend.import.DevFestNantes$getUrl$1.invokeSuspend(DevFestNantes.kt)

@ArtyMaury
Copy link
Copy Markdown
Contributor Author

@martinbonnin should be better now, I've changed on the Devfest repo side the path

@martinbonnin
Copy link
Copy Markdown
Collaborator

martinbonnin commented Jul 25, 2025

It now fails on:

java.lang.IllegalStateException: Cannot get https://raw.githubusercontent.com/GDG-Nantes/Devfest2025/main/data/categories.json: 404: Not Found
        at dev.johnoreilly.confetti.backend.import.DevFestNantes.getUrl(DevFestNantes.kt:130)
        at dev.johnoreilly.confetti.backend.import.DevFestNantes.access$getUrl(DevFestNantes.kt:115)

To debug this, you can run the import server locally with:

./gradlew localRun

And then run from another terminal:

curl -X POST http://localhost:8080/update/devfestnantes2025

You should see the errors in the first terminal. If you can make it so that it fails while saving the data to Google Datastore (this part needs credentials), then I can take it from there.

(edit: forgot the curl part)

@ArtyMaury
Copy link
Copy Markdown
Contributor Author

... a bit more refactoring than expected in our new website version, I'll fix all of that tonight

@martinbonnin
Copy link
Copy Markdown
Collaborator

Awesome, thanks!

@ArtyMaury
Copy link
Copy Markdown
Contributor Author

should be ok now, only gcp credentials missing in local mode so it reached the end 😃

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