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

Simpler asset loading #694

Closed
wants to merge 521 commits into from
Closed

Conversation

rowanseymour
Copy link
Contributor

No description provided.

Make the name of the ES index for contacts configurable
🔍 Use real elasticsearch in tests
Resetting elastic should re-index
Remove support for passing URNs to flow/preview_start as that's not a thing we do
Refresh elastic indexes after changes in tests instead of waiting for a second
Rework `search.ResolveRecipients` to use elastic
Remove applying started-previously exclusion in runner now that it's applied in batch creation
Cleanup functions use to load org assets
Add temp NOOP handler for `optin_created` events
Handle `optin_requested` events
Switch to using optin ids instead of UUIDs
Start switching to slog
Replace more logrus with slog
Use optin on broadcast to set `urn_auth` when queuing to courier
@github-actions github-actions bot locked and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants