Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

feat: step cyclic dimension with sn-action-button #454

Merged
merged 7 commits into from
Jan 3, 2024

Conversation

Bjornwa
Copy link
Contributor

@Bjornwa Bjornwa commented Dec 12, 2023

Add cyclic group action under feature flag.
Note: backend called is not working until engine release the new API.

@Bjornwa Bjornwa changed the title Call StepCycle feat: step cyclic dimension with sn-action-button Dec 12, 2023
@@ -576,7 +576,7 @@ describe("actions", () => {
isEnabled: jest.fn().mockReturnValue(true),
};
const result = getActionsList(shouldHide);
expect(result.length).toBe(20);
expect(result.length).toBe(21);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does unsupported feature navigations means? Is cyclic group an unsupported feature navigation?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for apps running in Direct Query mode. Unsupported features are controlled by engine and stored on the app layout. It would be good to verify if cyclic groups is something that can or cannot work in DQ apps

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. Björn have noted this in our test plan. I guess in the mean time we let this test be like this until we know it has support or not

@@ -585,7 +585,7 @@ describe("actions", () => {
isEnabled: jest.fn().mockReturnValue(true),
};
const result = getActionsList(shouldHide);
expect(result.length).toBe(13);
expect(result.length).toBe(14);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does feature blacklisted navigations means? is cyclic group one of the feature blacklisted navigations?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also for apps running in Direct Query mode. The difference with unsupported features is that this is controlled by the client. Mostly bigger parts that just do not make sense when the app is DQ

@linhnguyen-qlik linhnguyen-qlik marked this pull request as ready for review December 21, 2023 13:48
@linhnguyen-qlik linhnguyen-qlik requested a review from a team as a code owner December 21, 2023 13:48
@niekvanstaveren
Copy link
Collaborator

Maybe some tests would be nice

@linhnguyen-qlik
Copy link
Contributor

Maybe some tests would be nice

Tests added :)

@linhnguyen-qlik linhnguyen-qlik merged commit 311b679 into master Jan 3, 2024
5 checks passed
@linhnguyen-qlik linhnguyen-qlik deleted the cdz/feat-cyclic-groups-action branch January 3, 2024 08:46
qlikossbuild pushed a commit that referenced this pull request Jan 9, 2024
# [1.40.0](v1.39.0...v1.40.0) (2024-01-09)

### Features

* step cyclic dimension with sn-action-button ([#454](#454)) ([311b679](311b679))
@qlikossbuild
Copy link

🎉 This PR is included in version 1.40.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants