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

Improved: Move edit of invoice header to InvoiceSubTabBar (OFBIZ-12440) #409

Merged

Conversation

PierreSmits
Copy link
Member

In general, a TabBar menu is intended to give users access to overview related to the . And a SubTabBar menu is intended to give users to actions to be performed on the , like status changes, adding new records, etc.
The menu-item for editing the invoice header (aptly translated as 'Header') is currently located in the InvoiceTabBar menu in AccountingMenus.xml.
Given that this menu-item is intended for users with UPDATE permissions and to give those users access to a screen to edit the data, this menu-item must move to the InvoiceSubTabBar menu

Modified: AccountingMenus.xml

  • moved 'edit' menu-item for invoice from InvoiceTabBar menu to InvoiceSubTabBar menu. adjusted label.

In general, a <object>TabBar menu is intended to give users access to overview related to the <object>. And a <object>SubTabBar menu is intended to give users to actions to be performed on the <object>, like status changes, adding new records, etc.
The menu-item for editing the invoice header (aptly translated as 'Header') is currently located in the InvoiceTabBar menu in AccountingMenus.xml.
Given that this menu-item is intended for users with UPDATE permissions and to give those users access to a screen to edit the data, this menu-item must move to the InvoiceSubTabBar menu

Modified: AccountingMenus.xml
- moved 'edit' menu-item for invoice from InvoiceTabBar menu to InvoiceSubTabBar menu. adjusted label.
@PierreSmits PierreSmits changed the title Improved: Move edit of invoice header to InvoiceSubTabBar (OFBIZ012440) Improved: Move edit of invoice header to InvoiceSubTabBar (OFBIZ-12440) Dec 13, 2021
@sonarcloud
Copy link

sonarcloud bot commented Dec 13, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@JacquesLeRoux JacquesLeRoux merged commit bbe64fa into apache:trunk Jan 10, 2022
@PierreSmits PierreSmits deleted the OFBIZ-12440-Invoice-move-edit branch January 10, 2022 09:28
PierreSmits added a commit to PierreSmits/ofbiz that referenced this pull request Jan 10, 2022
* trunk: (80 commits)
  Improved: makes a French label consistent on the page
  Improved: Move edit of invoice header to InvoiceSubTabBar (OFBIZ012440) (apache#409)
  Improved: Have a Menu in Catalog featuring actions to create the main objects (OFBIZ-11774) (apache#433)
  Fixed: Store Group screen don't connect with grids (OFBIZ-12481) (apache#435)
  Improved: List and Grid (OFBIZ-11345) (apache#434)
  Improved: Use gradlew npm_outdated gradle-node-plugin taks to check outdated scripts (OFBIZ-12480)
  Improved: Use NPM with gradle to get external JS dependencies (OFBIZ-11960)
  Improved: List and Grid (OFBIZ-11345) (apache#432)
  Improved: List and Grid (OFBIZ-11345) (apache#431)
  Improved: VIEW permissions FinAccount mutations (OFBIZ-12451) (apache#410)
  Improved: Replace ELRTE plugin with Trumbowyg (OFBIZ-10093) (apache#425)
  Improved: Remove unused labels from AccountingUiLabels.xml (OFBIZ-9352) (apache#413)
  Improved: VIEW permissions BillingAccount (OFBIZ-12457) (apache#414)
  Improved: Invoice - CommonInvoiceDecorator (OFBIZ-12458) (apache#415)
  Improved: Agreement - CommonAgreementDecorator (OFBIZ-12462) (apache#418)
  Improved: List and Grid (OFBIZ-11345) (apache#426)
  Improved: Have a menu in Marketing featuring actions to create main objects (OFBIZ-11783) (apache#428)
  Improved: Have a Menu in Humanres featuring actions to create the main objects (OFBIZ-11777) (apache#429)
  Improved: Dutch translations (OFBIZ-10363) (apache#430)
  Documented: Gradle Wrapper is not downloaded with 18.12.03 (OFBIZ-12471)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants