Skip to content

Conversation

@anwesha-palit-redhat
Copy link
Contributor

@anwesha-palit-redhat anwesha-palit-redhat commented Nov 28, 2025

PR Description

This PR updates the time-range filter label from “Last weeks” to “Last week” to remove ambiguity and align with customer expectations. The associated API payload has also been updated to correctly send a 7-day range. This ensures consistency between UI behavior and backend interpretation.

Key Changes

  • Updated label in the time-range dropdown to “Last week”
  • Updated filter payload to use the correct single-week (1w) range

Outcome

The time-range filter is now clearer, intuitive, and accurately represents the selected time window in both UI and API interactions.

Screen Recording after change Tekton Result

SRVKP-9428.mov

Screen Recording after change Prometheus Metrics

SRVKP-9428_k8.mov

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Nov 28, 2025

@anwesha-palit-redhat: This pull request references SRVKP-9428 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.21.0" version, but no target version was set.

In response to this:

PR Description

This PR updates the time-range filter label from “Last weeks” to “Last week” to remove ambiguity and align with customer expectations. The associated API payload has also been updated to correctly send a 7-day range. This ensures consistency between UI behavior and backend interpretation.

Key Changes

  • Updated label in the time-range dropdown to “Last week”
  • Updated filter payload to use the correct single-week (1w) range

Outcome

The time-range filter is now clearer, intuitive, and accurately represents the selected time window in both UI and API interactions.

Screen Recording after change

SRVKP-9428.mov

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anwesha-palit-redhat

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [anwesha-palit-redhat]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Nov 28, 2025

@anwesha-palit-redhat: This pull request references SRVKP-9428 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.21.0" version, but no target version was set.

In response to this:

PR Description

This PR updates the time-range filter label from “Last weeks” to “Last week” to remove ambiguity and align with customer expectations. The associated API payload has also been updated to correctly send a 7-day range. This ensures consistency between UI behavior and backend interpretation.

Key Changes

  • Updated label in the time-range dropdown to “Last week”
  • Updated filter payload to use the correct single-week (1w) range

Outcome

The time-range filter is now clearer, intuitive, and accurately represents the selected time window in both UI and API interactions.

Screen Recording after change Tekton Result

SRVKP-9428.mov

Screen Recording after change Prometheus Metrics

SRVKP-9428_k8.mov

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@anwesha-palit-redhat
Copy link
Contributor Author

/retest-required

@arvindk-softwaredev
Copy link

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants