Skip to content

Xml generator 77 #141

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

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft

Xml generator 77 #141

wants to merge 15 commits into from

Conversation

dmitrytrager
Copy link
Collaborator

@dmitrytrager dmitrytrager commented Apr 10, 2025

What Issue Does This PR Cover, If Any?

This PR brings the logic that helps to generate and upload files with content for languages.
It will help to resolve these issues:
#77
#78
#80
#81

What Changed? And Why Did It Change?

  • Added fleet of generators for xml and text content
  • Added helper to write generated data into file and send file to proper destination
  • Added job that will manage content generation for every existing language
  • Added settings for recurring task that should trigger the job
  • Some tuneups for solid_queue, staging env configuration

How Has This Been Tested?

Rspec

Please Provide Screenshots

This feature does not include any interface except CLI

Additional Comments

We are moving forward several DBs for every env that we have.
This means we will have to specify DB every time we run some rake/rails task, for example.
Most of the times this will be primary DB

add processed file data structure
add logic to work with file writer and file sender
* main:
  reduce nesting in topics controller
  Bump brakeman from 7.0.0 to 7.0.2
  add new inputs to topic form, update specs
  move taggable concern to model concerns
  make by_year/by_month methods work with published_at
  fill published_at field in import data script
  save year and month into topic published_at new field
@dmitrytrager dmitrytrager force-pushed the xml-generator-77 branch 14 times, most recently from de19d34 to b5fec31 Compare April 12, 2025 13:40
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.

1 participant