search

Laravel-Backpack / community-forum

A workspace to discuss improvement and feature ideas, before they're actually implemented.
28 stars 0 forks source link

[Feature Request] Use forms with validation for Issue Request template #516

Open o15a3d4l11s2 opened 1 year ago

o15a3d4l11s2 commented 1 year ago

Feature Request

What's the feature you think Backpack should have?

Use a form for the bug reporting and feature request. Laravel uses this form for reporting bugs using an issue form.

Have you already implemented a prototype solution, for your own project?

Not yet, but here is the form template that Laravel uses.

Do you see this as a core feature or an add-on?

pxpm commented 1 year ago

Hey @o15a3d4l11s2 hope you are doing well.

It maybe be helpful indeed I agree with you.

At the moment we have the php artisan backpack:version that gives us automatically Laravel, PHP and Backpack package versions, you just copy paste the entire output and it doesn't look bad:

PHP VERSION:

PHP 8.2.5 (cli) (built: Apr 12 2023 08:43:43) (ZTS Visual C++ 2019 x64) Copyright (c) The PHP Group Zend Engine v4.2.5, Copyright (c) Zend Technologies with Xdebug v3.2.0, Copyright (c) 2002-2022, by Derick Rethans

LARAVEL VERSION:

10.2.0.0

BACKPACK PACKAGE VERSIONS:

backpack/basset: 0.15.0 backpack/crud: v6.x-dev backpack/medialibrary-uploaders: dev-main backpack/pro: v6.x-dev backpack/theme-tabler: dev-main

So I would probably create a form with 5 fields:

1 - Software versions (please do php artisan backpack:version and paste here the output). | REQUIRED 2 - What I am trying to do? (please describe what you are trying to achieve. | REQUIRED 3 - What happened? (please provide a descriptive error report. use images/videos etc to show the error) | REQUIRED 4 - Have you tried something to fix this? (maybe you already have the fix for it and want to share ?) (Can we make it required if answer is yes or something like that?) 5 - How can we reproduce this issue? (please provide reproduction steps that we can follow and get the same error as you) | REQUIRED

I think it will help us by having some required information on the bug reports. Most of the time we need to ask for more information, or spend some time trying to figure out how the error happens.

Making it a form also seems more "natural" than editing a textarea with some pre-defined template.

If @tabacitu agrees I think we should do this change.

Thanks for the suggestion @o15a3d4l11s2 🙏

Cheers

tabacitu commented 1 year ago

I love this! Thanks a lot @o15a3d4l11s2 . I'll take some time to do this next month - right now we're hyper-focused on fixing v6 bugs. If you'd enjoy working on this until then and would like to submit a PR @o15a3d4l11s2 , happy to merge.

@pxpm I would keep the questions and texts we have right now in our template - https://github.com/Laravel-Backpack/CRUD/issues/new?assignees=&labels=triage&projects=&template=bug_report.md&title=%5BBug%5D+ - they answer the same questions you've asked there but in a more intuitive format I think.