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

Dependency conflicts fix #42

Merged
merged 4 commits into from
Mar 26, 2021
Merged

Dependency conflicts fix #42

merged 4 commits into from
Mar 26, 2021

Conversation

nytr0gen
Copy link
Contributor

I really appreciate your work on this project.

I did notice the project wasn't working anymore because of some dependency conflicts. So I took it upon myself to to fix the dependencies by adding a Gemfile.lock and downgrading to ruby 2.3.8

I also made docker the main way of launching the project because it handles everything about the installation by itself, so it only requires one command and to have Docker installed. This was tested on Linux and Mac, but should work on the Windows version of Docker as well.

This should make it work practically forever without any new upgrade.

I made some updates to the project itself too:

  • Created an menu item for Help
  • Fixed the External OOB payload (at least I think so)
  • Sorted the generated by date DESC
  • Added volumes in docker-compose to have the database and generated persist
  • Removed the installation for ruby portion and only mentioned docker in the README. Did some formatting changes as well

Let me know if you agree with everything :D Suggestions are welcome

Copy link
Owner

@BuffaloWill BuffaloWill left a comment

Choose a reason for hiding this comment

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

Looks great @nytr0gen. I am going to merge and then re-archive the project.

I really appreciate the great effort and contribution back.

@BuffaloWill BuffaloWill merged commit 88b2a68 into BuffaloWill:master Mar 26, 2021
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