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

add dump_merger.sh script and sql needed to inserts on dump #76

Merged
merged 2 commits into from
Nov 7, 2022

Conversation

rasoro
Copy link
Member

@rasoro rasoro commented Oct 3, 2022

This PR is intended to provide a script to make it easy to create mailroom_test.dump from the nyaruka remote with whatever you need to keep in a fork, making conflict resolution easier with mailroom_test.dump.

requirements: docker, and bash

How to use:

After resolving the mailroom_test.dump conflict anyway, don't worry about the result, follow the next steps to make the dump ok.

  1. provide execution permission:

on repository top level directory run:

chmod +x dump_merger.sh
  1. run the script passing a MAILROOM_TAG variable with the tag from which you want to generate the reconciled dump:
MAILROOM_TAG=v7.1.22 ./dump_merger.sh

now you have a dump compatible with your fork.

obs: you can only run the script without variable if need to reconcile with the dump from nyaruka main.

when creating features with tests that need records in the dump in PR that will not be opened for nyaruka, or else that do, this script will be of great help.

@rasoro rasoro requested a review from Robi9 October 7, 2022 17:12
@rasoro rasoro changed the title add dump_merger.sh script and sql me needed inserts on dump add dump_merger.sh script and sql needed to inserts on dump Oct 7, 2022
@rasoro rasoro requested a review from matmsa27 November 7, 2022 15:09
@rasoro rasoro merged commit 73d8029 into main Nov 7, 2022
Robi9 added a commit that referenced this pull request Feb 2, 2024
Add handle for weniGPTCalled type events
rasoro pushed a commit that referenced this pull request Nov 26, 2024
🔍 Use real elasticsearch in tests
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.

3 participants