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

Use internal SQLite3 database #67

Merged
merged 32 commits into from
May 2, 2024
Merged

Use internal SQLite3 database #67

merged 32 commits into from
May 2, 2024

Conversation

vovimayhem
Copy link
Contributor

@vovimayhem vovimayhem commented Aug 14, 2022

[Closes #50]
[Closes #69]
[Closes #19]

Background

Replaces the PostgreSQL database with an internal SQLite database, as the data is typically read-only while running on servers, and updates can be deployed as part of the code. Also, it will fix some open issues.

Features

  1. Migrate DB from Postgresql to SQLite.
  2. Add an index to ZipCodes to full-text search inside SQLite
  3. Fix issues in searches caused by migrating between DBs
  4. add items per-page parameter to add flexibility when you require more than 15
  5. deployment has set from GCP to Azure, avoiding artifact registry, to save costs, now it uses Dockerhub
  6. No more PR deployments to save costs in the cloud
  7. upgrade DB from correos de México records

Conclusion

After those features, we got a renewed API to request the different Zip Codes inside Mexico.

@vovimayhem
Copy link
Contributor Author

The only thing not working that good is the "zip code" filtering functionality... here are some links with info related to how I'm planning to solve it:

Following the examples of the first link, I validated that looks like a viable replacement for the "unaccent" extension in PostgreSQL... but will deviate from the "standard" activerecord practice... meaning more SQL...

@Wikitelmex Wikitelmex marked this pull request as ready for review September 23, 2022 18:31
@github-actions github-actions bot temporarily deployed to feature/use-internal-database April 23, 2024 18:23 Destroyed
@github-actions github-actions bot temporarily deployed to feature/use-internal-database April 23, 2024 18:29 Destroyed
fix instruction in readme file

download the new zip codes database

add to rake task, the loadev task, to load locally the db
@github-actions github-actions bot temporarily deployed to feature/use-internal-database April 23, 2024 19:29 Destroyed
add dockerignore exception to /tmp/pids folder
@github-actions github-actions bot temporarily deployed to feature/use-internal-database April 24, 2024 15:25 Destroyed
@github-actions github-actions bot had a problem deploying to feature/use-internal-database April 24, 2024 17:08 Failure
@dvlex dvlex merged commit a9fa5af into main May 2, 2024
1 check failed
@dvlex dvlex deleted the feature/use-internal-database branch May 2, 2024 16:32
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.

More than 15 per page Remove the external database Calling City ID 3
4 participants