Skip to content

PDJB-302: Remove (hide) expired invites#1408

Draft
samyou-softwire wants to merge 3 commits into
feat/PDJB-300-property-record-landlord-invite-detailsfrom
feat/PDJB-302-remove-expired-jl-invites
Draft

PDJB-302: Remove (hide) expired invites#1408
samyou-softwire wants to merge 3 commits into
feat/PDJB-300-property-record-landlord-invite-detailsfrom
feat/PDJB-302-remove-expired-jl-invites

Conversation

@samyou-softwire
Copy link
Copy Markdown
Contributor

Ticket number

PDJB-302

Goal of change

allow for expired invites to be removed from the expired invites list

invites list with removed flash visible

Description of main change(s)

implemented this as a column on an invite - as I understand we should never permanently remove invites, for auditing reasons

adds a notification box based on a flash

adds a new status property to an invitation to make it clear how to use the flags and make it natural to exclude deleted invites

Checklist

Delete any that are not applicable, and add explanation below for any that are applicable but haven't been done

  • Screenshots of any UI changes have been added
  • Unit tests for new logic (e.g. new service methods) have been added
  • Controller tests for any new endpoints, including testing the relevant permissions
  • Test suite has been run in full locally and is passing
  • Branch has been rebased onto main and run locally, with everything working as expected (both for your new feature
    and any related functionality)
  • TODO comments referencing this JIRA ticket have been searched for and removed - if a future PR will address them,
    mention that here
  • QA instructions have been added to the ticket (particularly if this is the last PR required to complete the ticket)

@samyou-softwire samyou-softwire self-assigned this Jun 4, 2026
@samyou-softwire samyou-softwire changed the base branch from main to feat/PDJB-300-property-record-landlord-invite-details June 4, 2026 13:44
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