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.

Sign up for GitHub

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

Jump to bottom

Transaction Counter Improvements #3122

Open
6 of 7 tasks
ashanhr opened this issue Sep 24, 2024 · 4 comments · Fixed by wso2/apim-apps#766 · May be fixed by wso2/carbon-apimgt#12611
Open
6 of 7 tasks

Transaction Counter Improvements #3122

ashanhr opened this issue Sep 24, 2024 · 4 comments · Fixed by wso2/apim-apps#766 · May be fixed by wso2/carbon-apimgt#12611
Assignees
@Tharanidk
Labels
4.4.0 Component/APIM Type/Improvement
Milestone
APIM 4.4.0

Comments

@ashanhr
Copy link

ashanhr commented Sep 24, 2024 •
edited by Tharanidk
Loading

Problem

Following changes are suggested during the [DEMO] Transaction Counter Improvement.

Solution

  • Simplify configuration settings and change the configuration title in the transaction counter.
  • Update the permission logic to restrict transaction count visibility to other users. Only the super admin can view. Implementing a 403 response for different users. Other users should not be able to retrieve data by invoking an API.
  • UI
    • Adjust the UI layout to align title, buttons, and the transaction count.
    • Add a new button for viewing the data alongside the download button.
    • Rename the download button text from "Download PDF" to "Download Report."
    • Add a logo to the report and improve the appearance of the report.
    • Hide the UI components when they are disabled in the configuration settings.

Affected Component

APIM

Version

4.4.0

Implementation

No response

Related Issues

No response

Suggested Labels

No response
@Tharanidk Tharanidk self-assigned this Sep 24, 2024
@RakhithaRR RakhithaRR added this to the APIM 4.4.0 milestone Sep 24, 2024
@RakhithaRR
Copy link

Original feature: #3017

@Tharanidk
Copy link

Tharanidk commented Sep 25, 2024 •
edited
Loading

[work log]

following changes are done.

  • Rename the download button text from "Download PDF" to "Download Report."
  • Adjust the UI layout to align the title, buttons, and transaction count.
  • Improve the report's appearance, add the logo and a footer.
    Screenshot from 2024-09-25 13-35-30
    Screenshot from 2024-09-25 13-37-15

@Tharanidk
Copy link

Add a new button to view the data. After selecting the dates, click the view button to get the count.

Screenshot from 2024-09-25 16-52-27

@Tharanidk
Copy link

Added most of the configurations as default values and fixed the configuration title. Now the deployment.toml configurations should be defined as follows.

[apim.transaction_counter]
enable = true
server_id = "Gateway1"

This was referenced Sep 29, 2024
Merged
Open
@Tharanidk Tharanidk reopened this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tharanidk Tharanidk

Labels
4.4.0 Component/APIM Type/Improvement
Projects
None yet
Milestone
APIM 4.4.0
3 participants
@RakhithaRR @ashanhr @Tharanidk