Skip to content

chore: cleaner fetch paths for slugs - #371

Draft
bmschmidt wants to merge 1 commit into
mainfrom
01-11-cleaner_fetch_paths_for_slugs
Draft

chore: cleaner fetch paths for slugs#371
bmschmidt wants to merge 1 commit into
mainfrom
01-11-cleaner_fetch_paths_for_slugs

Conversation

@bmschmidt

@bmschmidt bmschmidt commented Jan 11, 2025

Copy link
Copy Markdown
Member

Suggested changes to dataset fetching. Goals are:

  1. Preserve current behavior;
  2. Add deprecation warning to the the behavior that depends on passing slugs without an org name, because this magic keeps biting people.
  3. Throw http errors directly instead of assuming things couldn't be found.

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@bmschmidt bmschmidt changed the title cleaner fetch paths for slugs chore: cleaner fetch paths for slugs Jan 11, 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.

1 participant