search

NASA-IMPACT / admg-backend

Apache License 2.0
2 stars 0 forks source link

When generating URLs for canonical models, use the snake_case version of the model name, rather than lowercase #569

Closed edkeeble closed 1 year ago

edkeeble commented 1 year ago

We expect model names to be snake case, but were generating links with the lowercase version.

Resolves #548

Tammo-Feldmann commented 1 year ago

This may not have been the best way to do this or to help you here but I opened a duplicate PR that has no merge conflicts and where the tests pass. Thought it would make your life easier but I realize that I might have created more of a mess by having two PRs. Sorry if this is more trouble than helpful @edkeeble.

edkeeble commented 1 year ago

All good. I just did the same thing for another PR :) Closing in favor of #573