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

chore(release): prepare source code for version 3.0.0 #58

Merged
merged 15 commits into from
Apr 25, 2024
Merged

Conversation

fabio-nettis
Copy link
Member

@fabio-nettis fabio-nettis commented Apr 25, 2024

Description

This PR contains all preperations for the new major V3 release. Following changes have been made in an effort to improve the api performance due to a noticable increase in traffic.

Overhauled docker build process 🐳

We modified the dockerfile to not only be much more readable but also to speed up build times by almost 50%. We moved the database location to the project root so we can create more database files as needed.

Fortified data refreshing ⏬

In an effort to make the data refreshing mechanism more reliable we refractored it to be a single transaction instead of many independent queries, this not only speeds up the process by about 200ms but also ensures that all data is available and intact when saved to the database.

SQLite based middleware 🐦

The rate limiting and request caching mechanisms now use SQLite as their storage startegy. This change was made in an effort to reduce the memory consumption. The main database as well as the new cache.db database files are located in the new database folder inside the project' root.

Reliable dates are a thing 📅

About time! With the help of the amazing Helldivers 2 Community we have figured out how to parse dates produced by the Arrowhead Game API and now provide accurate dates for Reports, Orders and every other content-type that has an in-game date.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Refractor (non-breaking change which cleans up code)
  • New feature (non-breaking change which adds functionality)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@fabio-nettis fabio-nettis added the feature Optimizes or introduces a feature label Apr 25, 2024
@fabio-nettis fabio-nettis merged commit 01c02bd into main Apr 25, 2024
3 checks passed
@fabio-nettis fabio-nettis deleted the feat/v3.0.0 branch April 25, 2024 12:18
Copy link

sentry-io bot commented Apr 27, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: Invalid value for parameter (id) parseIntParam(params.ts) View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Optimizes or introduces a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant