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

[NAE-2052] Integrate ticket view with menu items #269

Merged
merged 22 commits into from
Feb 18, 2025
Merged

Conversation

Retoocs
Copy link
Contributor

@Retoocs Retoocs commented Feb 14, 2025

Description

Implemented handling of new menu item configurations for DefaultTabbedTicketView and DefaultTabbedSingleTaskView

Implements NAE-2052

Dependencies

No new dependencies were introduced

Third party dependencies

No new dependencies were introduced

Blocking Pull requests

Depends on #261 and #267

How Has Been This Tested?

Manually and with tests

Test Configuration

Name Tested on
OS Ubuntu 24.04.1 LTS
Runtime Java 11
Dependency Manager Maven 3.6.3
Framework version Spring Boot 2.7.8
Run parameters
Other configuration

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @Kovy95
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

Retoocs and others added 20 commits January 28, 2025 10:03
- implement resolving of auto-select views in double-drawer
- abstract loading and navigation in double drawer menu
- implement default view opening in double drawer menu
- update tests according to changes
- add all nessesery public view components into engine default components
- add single task view and tabbed single task view to default components
- add directories
- copy of the files from project
# Conflicts:
#	projects/netgrif-components-core/src/lib/navigation/navigation-double-drawer/abstract-navigation-double-drawer.ts
- rework component resolving because of backend changes
- rename preference_item to menu_item
- update test for default-tab-view.component
- update imports in double-drawer-navigation.service
- add event emit in double-drawer-navigation.service
- add ticket view to default components and implement it as tabbed view
- implement handling of ticket view in double-drawer menu
@Retoocs Retoocs added the improvement New feature or request label Feb 14, 2025
@Retoocs Retoocs self-assigned this Feb 14, 2025
@Retoocs
Copy link
Contributor Author

Retoocs commented Feb 14, 2025

This test error is possibly from the branch I depend on. I will rerun tests as soon as it's fixed there
image

EDIT: it was fixed in the NAE-2041. I pulled the branch and none of the tests fail now

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
26.4% Coverage on New Code (required ≥ 50%)
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mazarijuraj mazarijuraj merged commit 5318b17 into release/6.5.0 Feb 18, 2025
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants