Skip to content

Conversation

cirdes
Copy link
Contributor

@cirdes cirdes commented Jul 27, 2024

New server: https://rbui.fly.dev

This PR includes the following updates and changes:

  1. Database Switch:

    • Removed PostgreSQL and Redis, and switched to SQLite3 for simpler web stack.
  2. Version Upgrades:

    • Updated Ruby to version 3.3.4.
    • Updated Rails to version 7.2.0.beta3.
  3. Asset Pipeline Update:

    • Removed Sprockets and added Propshaft for managing assets.
  4. Removed Unused Gems:

    • Removed the following unused gems:
      • stripe
      • letter_opener (development group)
      • postmark-rails (used in production)
      • octokit (version ~> 5.0)
      • faraday-retry (version ~> 2.0) - used for API calls failing in the octokit gem
      • rack-www - used to redirect www to non-www subdomain
      • rollbar
  5. Deployment Configuration:

    • Added configuration for deployment on Fly.io.

These changes streamline the project dependencies, simplify development, and prepare the application for deployment on Fly.io.

@cirdes cirdes changed the title Adding sqlite3 Switch to SQLite3, Upgrade Ruby and Rails, Remove Unused Gems, and Add Fly.io Deployment Configuration Jul 27, 2024
Copy link
Member

@sethhorsley sethhorsley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

almost perfect

@cirdes cirdes requested a review from sethhorsley July 29, 2024 15:03
@cirdes
Copy link
Contributor Author

cirdes commented Jul 29, 2024

@iseth, I have removed the ".tool-versions" file and fixed the comment in the Gemfile.

All other issues you mentioned were generated by the fly.io CLI, and I believe they are recommended by Yarn.

Let me know if you need anything else!

@sethhorsley
Copy link
Member

@iseth, I have removed the ".tool-versions" file and fixed the comment in the Gemfile.

All other issues you mentioned were generated by the fly.io CLI, and I believe they are recommended by Yarn.

Let me know if you need anything else!

I dig too deep into this but i think its definetly coming from https://github.com/fly-apps/dockerfile-rails/blob/2711cf5aab3716c3dafd1c08307ee62107a45dda/lib/generators/templates/_npm_install.erb

@cirdes
Copy link
Contributor Author

cirdes commented Jul 30, 2024

@iseth, I have removed .yarn/* and .yarnrc and added them to .gitignore.

Copy link
Member

@sethhorsley sethhorsley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome!

@sethhorsley sethhorsley merged commit dc14a12 into main Jul 30, 2024
@sethhorsley sethhorsley deleted the cirdes/move-to-sqlite-and-fly branch July 30, 2024 21:19
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