You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "PierreSmits (via GitHub)" <gi...@apache.org> on 2024/03/27 10:04:39 UTC
[PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
PierreSmits opened a new pull request, #741:
URL: https://github.com/apache/ofbiz-framework/pull/741
Integrate the overview of invoices past due of accounts receivable as a portal page intoo the main screen of the accounting application.
modified:
- AccountingUiLabels.xml: added label AccountingAccountsReceivable
- AccountingPortletData.xml: added PortalPortlet, PortalPage, PortalPageColumn and PortalPagePortlet
- CommonScreens.xml: added (for portal page handling): field setter, script and incude-portal-page in screen Main
- InvoiceForms.xml: added grid ListArReport
- InvoiceScreens.xml: added screen InvoicesArPastDue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "PierreSmits (via GitHub)" <gi...@apache.org>.
PierreSmits commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051155660
Hi Jacques,
Did you load the following from the changes in AccountingPortletData.xml into your local running instance and cleared the cache:
`
<PortalPortlet portalPortletId="InvoiceArPastDue" portletName="AR Invoices Past Due" screenName="InvoicesArPastDue"
screenLocation="component://accounting/widget/InvoiceScreens.xml" description="Invoices Past Due in Accounts Receivable"/>
<PortalPage portalPageId="ACC_MAIN" portalPageName="Accounting Main Page" description="The main portal page of the Accounting application" ownerUserLoginId="_NA_" sequenceNum="1"/>
<PortalPageColumn columnSeqId="00001" portalPageId="ACC_MAIN"/>
<PortalPagePortlet columnSeqId="00001" portalPageId="ACC_MAIN" portalPortletId="InvoiceArPastDue" portletSeqId="00001" sequenceNum="1"/>`
Without that the 'Could not find PortalPage with portalPageId [] is shown.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051165070
Must be that, trying, thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051127890
Hi Pierre,
After getting to https://localhost:8443/accounting/control/main just got this:
>java.lang.RuntimeException: Could not find PortalPage with portalPageId []
Just try :/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "PierreSmits (via GitHub)" <gi...@apache.org>.
PierreSmits commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2050002362
Indeed it has. Due to https://github.com/apache/ofbiz-framework/commit/c7333c0c93940a8bfeaab78e282a447755be8ab8 which happened at 12.11 today...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "sonarcloud[bot] (via GitHub)" <gi...@apache.org>.
sonarcloud[bot] commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051060148
## [](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 [0 Accepted issues](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_accepted_issues&view=list)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 No data about Coverage
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarCloud](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051247863
Indeed I remember now Leila's comments at OFBIZ-12918. How long this situation is expected to last?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "sonarcloud[bot] (via GitHub)" <gi...@apache.org>.
sonarcloud[bot] commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2022374284
## [](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 [0 Accepted issues](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_accepted_issues&view=list)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 No data about Coverage
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarCloud](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "PierreSmits (via GitHub)" <gi...@apache.org>.
PierreSmits commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2050407355
Merged the main branch into the PR branch. Conflicts resolved.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "sonarcloud[bot] (via GitHub)" <gi...@apache.org>.
sonarcloud[bot] commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2050325001
## [](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 [0 Accepted issues](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_accepted_issues&view=list)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_ofbiz-framework&pullRequest=741&resolved=false&inNewCodePeriod=true)
 No data about Coverage
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_ofbiz-framework&pullRequest=741&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarCloud](https://sonarcloud.io/dashboard?id=apache_ofbiz-framework&pullRequest=741)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "PierreSmits (via GitHub)" <gi...@apache.org>.
PierreSmits commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051208512
@JacquesLeRoux,
Great! (That it works).
RE: I guess you will put AP invoices list below AR?
There are many options available (e.g. AR portal pages besides AP portal pages, and 'due now' above 'due soon'.
However, I feel the community must decide how to move forward with this and the phasing out of the AR and AP applications. See also interactions in OFBIZ-12918
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051191253
I fastly tried to show something more helping than
> Could not find PortalPage with portalPageId []
in those cases, it's too long
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2049929319
We got some conflicts here
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051176320
Works, I feel the main page makes more sense now. I guess you will put AP invoices list below AR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051306917
The situation about Leila's question:
>I was warning you because I notice the broken screen when doing tests for my last Jira. I propose to add it just to repair the screen during the time that Jiras [OFBIZ-10552](https://issues.apache.org/jira/browse/OFBIZ-10552) and [OFBIZ-12966](https://issues.apache.org/jira/browse/OFBIZ-12966) are managed.
Is it ok for you ?
For the rest
1. Where adding the AR andAP "invoices due screens" in accounting;
2. then removing AR and AP components
we could resurrect, but better create a last thread to get a consensus
What do you think?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux merged PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improved: Integrate AR Past Due into accounting-main (OFBIZ-12966) [ofbiz-framework]
Posted by "PierreSmits (via GitHub)" <gi...@apache.org>.
PierreSmits commented on PR #741:
URL: https://github.com/apache/ofbiz-framework/pull/741#issuecomment-2051278241
Which situation?
And in general: ask nobody. Nobody knows... ;)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org