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

[Formie -> forms] add custom status field to form and allow for filtering in backend #1502

Open
wants to merge 1 commit into
base: craft-4
Choose a base branch
from

Conversation

Numkil
Copy link
Contributor

@Numkil Numkil commented Jul 17, 2023

Hiya,

This PR is in relation to issue #1471

I implemented the simplest way I could think off in my short craft experience to add some very basic statuses on Form level. I would be more than happy to work on this a lot more to rewrite this in a more solid way.

This way we can at least offer our clients the option to manage their forms a bit more conveniently when they have a large amount of outdated forms that they do not want to delete for administrative reasons.

Numkil

@Numkil
Copy link
Contributor Author

Numkil commented Oct 5, 2023

Hiya @engram-design

Was wondering if you would ever be interested in considering this addition to the codebase. As i've said above, im more than happy to flesh this out more if this basic approach is not satisfactory.

@engram-design
Copy link
Member

engram-design commented Oct 5, 2023

I’m still debating whether to merge this or not, as the long term goal will be to implement Form Groups #1039 and as such, adding a feature for the short-term that will likely be removed isn’t something I’m keen on.

I’m undecided whether we want to have both features.

@Numkil
Copy link
Contributor Author

Numkil commented Oct 5, 2023

Thank you for the information, thats good enough for me! :) you know where to find me if you'd like me to revisit this.

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