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

[IMP] l10n_it_account_stamp: Usability for stamp in invoice. #3452

Conversation

TonyMasciI
Copy link
Contributor

If stamp line can be added (invoice in draft): show button to add stamp line. If stamp line can't be added (invoice not in draft): show message explaining why it can't be added. If stamp line has already been added: show message saying it has already been added (useful if invoice has many lines). Button and messages update live while updating the invoice. Better messages to user for charging stamp to customer

If stamp line can be added (invoice in draft): show button to add stamp line.
If stamp line can't be added (invoice not in draft): show message explaining why it can't be added.
If stamp line has already been added: show message saying it has already been added (useful if invoice has many lines).
Button and messages update live while updating the invoice.
Better messages to user for charging stamp to customer
@TonyMasciI TonyMasciI marked this pull request as ready for review July 7, 2023 09:18
@francesco-ooops francesco-ooops linked an issue Dec 19, 2023 that may be closed by this pull request
3 tasks
@francesco-ooops
Copy link
Contributor

@TonyMasciI puoi risolvere i conflitti?

Copy link

github-actions bot commented Jun 2, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 2, 2024
@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@francesco-ooops The rebase process failed, because command git rebase origin/14.0 failed with output:

Rebasing (1/1)
Auto-merging l10n_it_account_stamp/models/account_move.py
Auto-merging l10n_it_account_stamp/views/account_move_view.xml
CONFLICT (content): Merge conflict in l10n_it_account_stamp/views/account_move_view.xml
error: could not apply e79357a56... [IMP] l10n_it_account_stamp: Usability for stamp in invoice.
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply e79357a56... [IMP] l10n_it_account_stamp: Usability for stamp in invoice.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 14, 2024
@SirAionTech
Copy link
Contributor

Chiudo per inattività, sostituita da #4414.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants