Skip to content

feat: toggle database seeds via settings page#11

Open
mithereal wants to merge 2 commits intopuemos:mainfrom
data-twister:seed_updates
Open

feat: toggle database seeds via settings page#11
mithereal wants to merge 2 commits intopuemos:mainfrom
data-twister:seed_updates

Conversation

@mithereal
Copy link
Copy Markdown
Contributor

cleanup seeds, add button in settings to seed/delete demo data

add release seeder, add button in settings to seed demo data

if System.get_env("PHX_SERVER") do
if System.get_env("PHX_SERVER") ||
System.get_env("RAILWAY_PUBLIC_DOMAIN") do
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

let's try to avoid specific platforms names because if we don't this list will grow quickly

host = System.get_env("HOST") || System.get_env("PHX_HOST") || "example.com"
host =
System.get_env("HOST") || System.get_env("PHX_HOST") ||
System.get_env("RAILWAY_PUBLIC_DOMAIN") ||
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Same, avoid specific platform names

{:open_api_spex, "~> 3.16"},
{:req, "~> 0.5", only: [:dev, :test]}
{:req, "~> 0.5", only: [:dev, :test]},
{:glific_phil_columns, github: "data-twister/phil_columns-ex"},
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I'd like to avoid using a library for that, can you bring this in-house or use a more stable and well known library?

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.

i switched to phil-columns bc its the best i know of that does seeding across multi files, i generally clone into this repo and use the forks in my projects so they dont dissapear, also i normally dont include the migrations and repo/resource_snapshots electing to use mix create during the deploy. just my pref.

</section>
</div>
<section
id="forecasting-settings"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

need a better id

</section>

<section
id="forecasting-settings"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

need a better id

@puemos
Copy link
Copy Markdown
Owner

puemos commented Mar 19, 2026

Can we try to split this into PR per feature, there are a few combined. this will make it simpler to rollback if needed. I think one PR per fix + one PR for the seed system.
Thanks!

split featureset into smaller chunks
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