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

idea: make "services" rows editable #512

Open
TreyWW opened this issue Oct 6, 2024 · 5 comments
Open

idea: make "services" rows editable #512

TreyWW opened this issue Oct 6, 2024 · 5 comments
Assignees
Labels
idea: suggested This idea has been suggested. Awaiting recognition. idea

Comments

@TreyWW
Copy link
Owner

TreyWW commented Oct 6, 2024

How big is this feature?

Small

Describe the problem

At the moment services are tied to the modal and can only be deleted.

Describe a possible solution

Instead of modals just use inputs so its easy to edit, no need for modals

@TreyWW TreyWW added idea idea: suggested This idea has been suggested. Awaiting recognition. labels Oct 6, 2024
@Parkerrino
Copy link

Hey, I would like to look into that could you assign it to me pls? ^^

@TreyWW
Copy link
Owner Author

TreyWW commented Nov 26, 2024

Hey @Parkerrino sure! Do you know when you may be able to start working on this? Let me know if you need any help getting started :)

@Parkerrino
Copy link

Hey@TreyWW, thx for the quick response, once I have a slight idea of the code, I could start right away :)

@TreyWW
Copy link
Owner Author

TreyWW commented Nov 26, 2024

Great thanks! Once cloned follow the docs from here.

If you go to "Invoices" then create, this is the part this issue is talking about:

image

Basically instead of this modal v it'll be inline editable rows and "add service" would just create a row.
image

It'll also need validation on the frontend.

Do let me know if you have any questions or need help getting setup. Really appreciate your quick reply.

@Parkerrino
Copy link

for now it looks like this, I also could try to make it, that it always opens the edit menu for the latest invoice
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea: suggested This idea has been suggested. Awaiting recognition. idea
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants