api-manager icon indicating copy to clipboard operation
api-manager copied to clipboard

Transaction Counter Improvements

Open ashanhr opened this issue 1 year ago • 4 comments

Problem

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

Solution

  • [x] Simplify configuration settings and change the configuration title in the transaction counter.
  • [x] 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
    • [x] Adjust the UI layout to align title, buttons, and the transaction count.
    • [x] Add a new button for viewing the data alongside the download button.
    • [x] Rename the download button text from "Download PDF" to "Download Report."
    • [x] 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

ashanhr avatar Sep 24 '24 04:09 ashanhr

Original feature: https://github.com/wso2/api-manager/issues/3017

RakhithaRR avatar Sep 24 '24 16:09 RakhithaRR

[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 avatar Sep 25 '24 08:09 Tharanidk

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 avatar Sep 25 '24 11:09 Tharanidk

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"

Tharanidk avatar Sep 26 '24 11:09 Tharanidk