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

Improve README #14

Closed
lbraits opened this issue Aug 2, 2024 · 4 comments
Closed

Improve README #14

lbraits opened this issue Aug 2, 2024 · 4 comments

Comments

@lbraits
Copy link

lbraits commented Aug 2, 2024

Hi,

First of all, thank you for this extension! We use it in a self-hosted Directus instance for the export/import feature flow, and it works pretty well for us.

In the last few weeks, we have noticed a recurring warning in our logs that seems to have appeared after an update of the extension:

[01:00:01.041] WARN: update "directus_flows" set "flow_manager_last_run_at" = $1, "flow_manager_run_counter" = $2 where "id" = $3 - column "flow_manager_last_run_at" of relation "directus_flows" does not exist

After reading through some of the extension's code, we have figured out that these missing columns must be created manually via the Configure button on the extension's main page. I think it would be helpful to have this documented in the README as a required step when installing or updating the extension. Additionally, I am wondering if it would be possible to automate this process of migrating the database, for example, via a hook on one of Directus's init events, like the 'app.before' event?

Thank you!

@baguse
Copy link
Owner

baguse commented Aug 3, 2024

Hi, thanks for trusting my extension and sorry for the trouble. Yes, I already noticed the extension problem. I already add the warning message on the instance, should I make the field creation automatically?

@lbraits
Copy link
Author

lbraits commented Aug 5, 2024

Hi,
I think a warning or info would be nice, but automatic field creation would be even nicer :)

@baguse
Copy link
Owner

baguse commented Aug 5, 2024

Hi, I think a warning or info would be nice, but automatic field creation would be even nicer :)

Ok then, thanks for the input

@baguse
Copy link
Owner

baguse commented Aug 7, 2024

Hi @lbraits, I fixed this issue on v1.4.5. Kindly check

@baguse baguse closed this as completed Aug 7, 2024
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

No branches or pull requests

2 participants