Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

on call: fix search on stage #8743

Merged
merged 3 commits into from
Jun 6, 2024
Merged

on call: fix search on stage #8743

merged 3 commits into from
Jun 6, 2024

Conversation

alecsavvy
Copy link
Contributor

Description

there weren't defaults set for these fields if they don't exist which causes python to raise an error

How Has This Been Tested?

testing on dn5 by execing into container

@alecsavvy alecsavvy requested a review from dylanjeffers June 6, 2024 15:24
@alecsavvy alecsavvy self-assigned this Jun 6, 2024
Copy link

changeset-bot bot commented Jun 6, 2024

⚠️ No Changeset found

Latest commit: 2943352

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@alecsavvy alecsavvy requested a review from isaacsolo June 6, 2024 16:56
@pull-request-size pull-request-size bot added size/S and removed size/XS labels Jun 6, 2024
@alecsavvy alecsavvy enabled auto-merge (squash) June 6, 2024 17:15
@alecsavvy alecsavvy merged commit dfb04fd into main Jun 6, 2024
12 checks passed
@alecsavvy alecsavvy deleted the as/fix-search branch June 6, 2024 17:26
alecsavvy added a commit that referenced this pull request Jun 6, 2024
schottra added a commit that referenced this pull request Jun 6, 2024
…-gated-content

* origin/main: (24 commits)
  [C-4467] Fix path reset on search (#8739)
  [C-4263] Add OptionsFilterButton and implement more search filters (#8737)
  [C-4660] POC: Pre-fetch CN stream urls (no /stream redirects) (#8741)
  on call: fix search on stage (#8743)
  Audius Protocol v0.6.120
  [PAY-2835] Add account switcher to oauth popup (#8716)
  [PAY-3086] Add verified user rate limit (#8729)
  [PAY-3085] Move payout wallet radio input to labeled (#8730)
  Add test to ensure updating playlist_id fails (#8736)
  [QA-1338] Fix headers for small screens (#8732)
  [C-4466] Update new search results page track preview number (#8735)
  [QA-865] Fix track mood alignment (#8731)
  [C-4443][C-4444] Scheduled album releases (#8667)
  [C-4361] Add grid and list layouts for new search track view (#8733)
  Fix lint (#8734)
  [C-4463] POC add a 'skip check' param to /stream (#8709)
  [QA-1306] Fix mobile text-link finalized state (#8720)
  [QA-1287] Fix multiple popovers on tracks table (#8721)
  [QA-1346] Fix select-page dropdown text alignment (#8722)
  [C-4360] Update SearchBarV2 to search on input (#8726)
  ...
audius-infra pushed a commit that referenced this pull request Jun 7, 2024
[fa19828] [PAY-3091] Move social handles out of identity into dn via entity manager (#8728) Saliou Diallo
[fe60b30] Fix dupe swagger id (#8745) JD Francis
[ed8febd] [C-4455] Update otp email (#8727) Isaac Solo
[8496557] [C-4263] Add OptionsFilterButton and implement more search filters (#8737) Sebastian Klingler
[49b5a56] [C-4660] POC: Pre-fetch CN stream urls (no /stream redirects) (#8741) JD Francis
[dfb04fd] on call: fix search on stage (#8743) alecsavvy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants