New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add contact archiving #1769
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments, otherwise AWESOMMMME.
CHANGELOG
Outdated
@@ -14,6 +14,7 @@ UNRELEASED CHANGES: | |||
* Fix gift recipient relation | |||
* Fix subscription cancel on account deletion | |||
* Fix email maximum size on settings | |||
* Add contact archiving |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add ability to archive a contact
<div> | ||
<notifications group="archive" position="top middle" duration=5000 width="400" /> | ||
|
||
<a href="#" class="btn edit-information" @click="toggle">{{ active ? $t('people.contact_archive') : $t('people.contact_unarchive') }}</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can simply omit href="#"
entirely.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one more tiny change 😀
resources/lang/en/people.php
Outdated
@@ -63,6 +63,7 @@ | |||
'edit_contact_information' => 'Edit contact information', | |||
'contact_archive' => 'Archive contact', | |||
'contact_unarchive' => 'Unarchive contact', | |||
'contact_archive_help' => 'Archived contact will not appear on contact lists', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Archived contacts will not be shown in the contact list
Don't forget to run Cypress on this one. |
@djaiss cypress fails on my computer on master branch: I cannot validates any modification with it. |
I've rewritten some of the screens, also removed the button and use a link instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request has been automatically locked since there |
This close #1623
(Add api functions to get archived contacts => this one will be done in another PR)