Skip to content

PROTO-1781: move verified notifs into pedalboard + ci - #9130

Merged
alecsavvy merged 7 commits into
mainfrom
as/proto-1781-verified-notifs-ci
Jul 16, 2024
Merged

PROTO-1781: move verified notifs into pedalboard + ci#9130
alecsavvy merged 7 commits into
mainfrom
as/proto-1781-verified-notifs-ci

Conversation

@alecsavvy

@alecsavvyalecsavvy commented Jul 15, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Description

  • moves verified bot into pedalboard correctly
  • uses pedalboard ci to build image
  • can run plugin locally with audius-compose up verified-notifications for testing
  • removes old build scripts and stuff from pre pedalboard

How Has This Been Tested?

see commit here where build was successful
9e0a778
audius-compose up verified-notifications connects correctly to localstack

@changeset-bot

changeset-botBot commented Jul 15, 2024•
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 88dcd91

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

@alecsavvyalecsavvy changed the title PROTO-1781: verified notifs ciPROTO-1781: move verified notifs into pedalboard + ciJul 15, 2024
@audius-infra

audius-infra commented Jul 15, 2024

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/as/proto-1781-verified-notifs-ci

@audius-infra

audius-infra commented Jul 16, 2024

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/as/proto-1781-verified-notifs-ci

@alecsavvy
alecsavvy marked this pull request as ready for review July 16, 2024 16:52
@audius-infra

audius-infra commented Jul 16, 2024

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/as/proto-1781-verified-notifs-ci

@alecsavvy
alecsavvy enabled auto-merge (squash) July 16, 2024 17:22
michellebrier
michellebrier approved these changes Jul 16, 2024
View reviewed changes

@michellebriermichellebrier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@alecsavvy
alecsavvy merged commit 15b031c into mainJul 16, 2024
@alecsavvy
alecsavvy deleted the as/proto-1781-verified-notifs-ci branch July 16, 2024 21:42
audius-infra pushed a commit that referenced this pull request Jul 17, 2024
audius-infra
Audius Protocol v0.6.149
de5b102
[ffd2ad3] [PAY-2783] Add filters to manager endpoints (#9109) Randy Schott
[dc387b0] [PAY-3166] Updates ESLint dependencies to fix broken IDE experience (#9146) Randy Schott
[bda6e63] Specifiers should be strings in challenge notifs (#9148) Raymond Jacobson
[96efa17] Cycle solana RPC connections better (#9147) Raymond Jacobson
[3d5ff3a] [C-4777] Prevent public track with future release date (#9139) Reed
[faf7908] Final touches to UDR (#9144) Raymond Jacobson
[15b031c] PROTO-1781: move verified notifs into pedalboard + ci (#9130) alecsavvy
[8d442d3] Fix misc UDR bugs (#9141) Raymond Jacobson
[1183296] PROTO-1879: rm PUT /user/email route (#9089) alecsavvy
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raymondjacobsonraymondjacobsonAwaiting requested review from raymondjacobson
1 more reviewer
@michellebriermichellebriermichellebrier approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alecsavvy@audius-infra@michellebrier