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

[ECO-2111] Fix start version behavior, add web_anon postgrest user #41

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

xbtmatt
Copy link

@xbtmatt xbtmatt commented Aug 22, 2024

  • Add web_anon postgrest user
  • Change config.rs starting version field name to: MINIMUM_STARTING_VERSION instead of STARTING_VERSION
  • Use the max(a, b) value of MINIMUM_STARTING_VERSION from the config.yaml file field value vs. the processor's latest processed version
  • Note this means we have to change the compose.yaml file to pass in MINIMUM_STARTING_VERSION instead of STARTING_VERSION
  • This removes the bash code logic from having to query the database (since it's actually already done in the indexer) and uses a clearer env var name to indicate what the field actually does
  • To summarize: we now take the minimum value between the config starting version and the latest processed starting version. It's determined entirely inside the processor with no logic in a bash script (which was prone to errors before).

@xbtmatt xbtmatt requested review from alnoki and CRBl69 August 22, 2024 07:21
@alnoki alnoki merged commit 94e6788 into emojicoin-dot-fun Aug 22, 2024
1 of 4 checks passed
@alnoki alnoki deleted the ECO-2111 branch August 22, 2024 19:53
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.

2 participants