Skip to content

fix: display all actions in lifecycle rules - #242

Closed
cxymds wants to merge 1 commit into
mainfrom
cxymds/fix/lifecycle-rule-display-clean
Closed

cxymds wants to merge 1 commit into
mainfrom
cxymds/fix/lifecycle-rule-display-clean

Conversation

@cxymds

@cxymds cxymds commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Fix lifecycle rule rendering when one S3 rule contains both current-version and non-current-version actions. The UI now projects every lifecycle action and renders each version/duration pair in desktop tables and mobile cards, instead of pairing the non-current label with Expiration.Days.

Type of Change

  • Bug fix
  • Test improvements

Testing

  • Unit tests added/updated
  • Manual testing completed (RustFS API was unavailable locally; the UI was verified through the action projection tests)

Passed locally:

pnpm type-check
pnpm lint
pnpm format:check
pnpm test:run

Related Issues

Closes #8036

Screenshots

The screenshots use the issue's two-rule payload (30/40 days and 10/20 days) and show the expected post-fix desktop and mobile layouts.

Desktop lifecycle actions

Mobile lifecycle actions

Additional Notes

The original S3 rule remains the deletion unit; the projected action list is display-only, so hidden lifecycle fields are not lost when a rule is deleted.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant