-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
FIX - Issue 304. #478
FIX - Issue 304. #478
Conversation
Awaiting filter removal from template |
removed filters |
Removed migration file |
Apologies for the delay. If possible today I will try the changes locally make sure it works as expected on a few devices, and hopefully get this merged! Thanks for your patience, I appreciate the contribution |
ok, got it |
Yep seems to work well on mobile and desktop. Thanks for your first contribution @pvvramakrishnarao234, much appreciated! Let me know on discord if there's anything further you'd like to work on, there's a few interesting ideas i'll be implementing in the next few weeks so it's a good time to spot in :) |
Description
Added a condition at /invoices/single so that last two invoices have a dropup and not go behind the parent container.
Checklist
djLint-er on any new code
(checks will fail without)
changes
What type of PR is this?
Added/updated tests?
Related PRs, Issues etc