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

Implement naive feed configuration storage #1399

Draft
wants to merge 8 commits into
base: 5.x
Choose a base branch
from

Conversation

lthurston
Copy link

@lthurston lthurston commented Jan 14, 2024

Description

This is the beginning of an idea to read / write feed configuration from / to the filesystem. This allows feeds to be tracked in the filesystem, like any other configuration or code. Developers will be able to create the feeds locally, write the feed to the filesystem using ./craft feed-me/feed-config-storage/write, commit that file, and, as a deploy step, read the configuration from disk with ./craft feed-me/feed-config-storage/read.

This hasn't been fully tested. I assume there are glaring bugs. Is this a feature that might be considered valuable enough to be incorporated into the plugin, or are we unique in this need? We can, but prefer not to, maintain our own fork, but I'd rather share this work with others.

I'm curious to hear your thoughts and whether you can offer any course corrections or other feedback. Thanks!

Note: you will need to mkdir config/feed-me for this to work at the moment.

@tricki
Copy link

tricki commented Feb 15, 2024

FYI I manually extracted some feeds but ran into some problems when deploying to different servers with different databases. The main problem was the fact that the configuration contains auto-generated IDs (e.g. siteId), which might be different on different deployments. In my current solution I still have to configure those manually (site, section type and entry type, if I remember correctly).

This lets us do things like put the current
date into the feedUrl.
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.

None yet

3 participants