Skip to content

[FFS-4372] introduce simple rails app poc#70

Merged
iannorriswork merged 7 commits into
mainfrom
rails-app-poc
May 13, 2026
Merged

[FFS-4372] introduce simple rails app poc#70
iannorriswork merged 7 commits into
mainfrom
rails-app-poc

Conversation

@iannorriswork
Copy link
Copy Markdown
Contributor

@iannorriswork iannorriswork commented May 5, 2026

Plan:

  • merge this code in as is, allowing either version to be deployed (Go or Rails version)
  • deploy and test in dev to confirm that this works as expected
  • run a followup PR that puts the rails code as priority and removes the Go code
  • run a followup PR to finish up any shoryuken required changes and infra
  • followup PR to add static analysis

questions:

  • is it convenient to leave this in a Rails directory rather than have the Rails app be top level?

Comment thread rails-app/config/credentials.yml.enc Outdated
@@ -0,0 +1 @@
yFV6DSRoTZ1pdllYlXsMDP2bUj8XDOn7cRZk2C3yVWFg42y2zhLzXv7A1UAaDBpfvnZBUVKTUV5ZQmV1GZi/Z/wjyvp49fi1CY6h8IM1YH7Q8w1Itly6eQUFvkgpW6XIj6njAcYF+xNpbDeLjF92rpzH0zIfhT/iSr9HrqIL8umHBMT2o8M9a9FBx3j1xViB0GBzdAwfojHDgGF7IiqGRB/B3tIStzxDhRRas1UNQRbBdjfHpCKfHn+ygsLK3zIKYQxx0K6kbyUqTYVIu7XPWOzgbHVqaOtwSyBLUinaoOKeNTzHm3gwmdwE7XcKs4aO09R0k6+rzpjmO3gY3Ntfzxzvhxy/CRwzIeEyew76lVgEmCjJbbQMZ9jE8er4/A2OMkrX0CenFRVbmhavioK0I/JhKGOXd5rNA1qIBbJp4ggPn5BWinf1xKFBSgT9Lr8SmT9af+UPNyGYeIhklRA1SzjK2nfMSjZZQb5p9wnWtA+rXo7g7yuJXJul--zlXm+2yPJdzAGfjC--w8rQn9sba6VZ2SgCj9xehA== No newline at end of file
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we'll probably favor heavy environment variables over encrypted rails credentials, given the open source nature of things.

@iannorriswork iannorriswork changed the title introduce simple rails app poc [FFS-4372] introduce simple rails app poc May 12, 2026
@iannorriswork iannorriswork requested a review from a team May 13, 2026 13:10
Copy link
Copy Markdown
Contributor

@jeffgreenland-cms jeffgreenland-cms left a comment

Choose a reason for hiding this comment

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

Overall looks thorough from my review.

@iannorriswork
Copy link
Copy Markdown
Contributor Author

Going to merge as in, but feel free to comment for necessary followup or questions.

@iannorriswork iannorriswork merged commit 59c6021 into main May 13, 2026
5 checks passed
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