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.

Sign up for GitHub

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

Jump to bottom

fix ddex playlist release_date parsing #8421

Merged
merged 1 commit into from
May 9, 2024
Merged

fix ddex playlist release_date parsing #8421

merged 1 commit into from
May 9, 2024

Conversation

michellebrier
Copy link
Contributor

Description

Fix so indexing doesn't skip this tx. Still need to debug why this field is missing in the metadata (I can see it in the _metadata in the healthz tx explorer) but this makes testing a fix easier.

How Has This Been Tested?

deployed to a stage node, verified tx succeeded

@michellebrier michellebrier requested a review from isaacsolo May 9, 2024 22:12
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented May 9, 2024

⚠️ No Changeset found

Latest commit: 9042229

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

@michellebrier michellebrier merged commit 743215b into main May 9, 2024
11 checks passed
@michellebrier michellebrier deleted the mbrier/proto-1797/fix-ddex-rd branch May 9, 2024 22:15
audius-infra pushed a commit that referenced this pull request May 10, 2024
Audius Protocol v0.6.100
a235686 
[e9a66b1] [QA-1259] Restrict OTP spam (#8426) Dylan Jeffers
[743215b] fix ddex playlist release_date parsing (#8421) Michelle Brier
[b21ec67] solana relay ip utils (#8415) alecsavvy
schottra added a commit that referenced this pull request May 10, 2024
@schottra
Merge remote-tracking branch 'origin/main' into pay-2932-cleanup-apic…
d594eb6 
…lient

* origin/main: (25 commits)
  [QA-1268] Fix sentry upload (#8432)
  [QA-1253] Improve details tile layout (#8433)
  [QA-1251] Fix short lineup playback issues. (#8402)
  fix ddex playlists UI (#8435)
  [PAY-2965, PAY-2968, PAY-2963] Album upload copy fixes; remove 'hidden' option (#8428)
  [PAY-2786] Clean up skeletons for collection page (#8427)
  Fix dog ear pt 3 (#8420)
  Audius Protocol v0.6.100
  QA-1260: change content title to not say track (#8436)
  [Web] Implement Remove Manager flows [Manager Mode ] [PAY-2827] (#8413)
  [Web] [Common] Accounts You Manage flows; hook up Accounts Managing You to API (#8301)
  [QA-1267] Fix blank android tip screen (#8430)
  [QA-1259] Restrict OTP spam (#8426)
  [PAY-2985] Fix squished art in purchase modal (#8423)
  [PAY-2960] Change library filter title to Premium (#8422)
  fix ddex playlist release_date parsing (#8421)
  Fix web-test by resolving circular import (#8418)
  solana relay ip utils (#8415)
  Revert "Fix dog ear on track page" (#8416)
  Misc $album fixes (#8393)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alecsavvy alecsavvy alecsavvy approved these changes

@isaacsolo isaacsolo isaacsolo approved these changes

@raymondjacobson raymondjacobson raymondjacobson approved these changes

Assignees
No one assigned
Labels
size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@michellebrier @raymondjacobson @isaacsolo @alecsavvy