Skip to content

swipe actions have been hidden for the Local Account - #344

Merged
suraj-yadav0 merged 1 commit into
CITOpenRep:release/v1.3.3from
parvathyabnair:localaccount/hide
Sep 10, 2026
Merged

suraj-yadav0 merged 1 commit into
CITOpenRep:release/v1.3.3from
parvathyabnair:localaccount/hide

Conversation

@parvathyabnair

Copy link
Copy Markdown
Contributor

Summary

The swipe actions have been hidden for the Local Account

Type

  • Feature
  • Bug Fix
  • Refactor
  • Documentation
  • Performance
  • Testing

Motivation

Changes

1.leadingActions: Evaluates to model.id === 0 ? null : accountLeadingActions, hiding the Edit action when swiping left on the Local Account.
2.trailingActions: Evaluates to model.id === 0 ? null : accountTrailingActions, hiding the Log and Delete actions when swiping right on the Local Account.
3.onClicked: Retains if (model.id !== 0) so clicking the Local Account row does not attempt to open the remote account configuration page.

Testing

  • Tested on Emulator (clickable build && clickable review)
  • Tested on device (clickable install)
  • Unit tests added/updated
  • No existing Functionality/Tests broken

Screenshots (if UI change)

Before After

Checklist

  • Code follows existing style conventions
  • No hardcoded credentials or secrets
  • No debug logs left in production code
  • Documentation updated (if applicable)
  • Squash commits before review (if needed)
  • PR is up-to-date with main or release\x.x.x branch

Related Issues

Closes #

Additional Notes

#339

@suraj-yadav0 suraj-yadav0 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@suraj-yadav0
suraj-yadav0 merged commit 3286e00 into CITOpenRep:release/v1.3.3 Sep 10, 2026
1 check passed
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.

2 participants