You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ponymail.apache.org by gi...@git.apache.org on 2016/05/31 11:13:45 UTC

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

GitHub user  opened a pull request:

    https://github.com/apache/incubator-ponymail/issues/59

    New feature: allow admins to remove e-mails from archives

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/incubator-ponymail 

Alternatively you can review and apply these changes as the patch at:

    

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #59
    
----

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail issue #59: New feature: allow admins to remove e-mails fr...

Posted by ilgrosso <gi...@git.apache.org>.
Github user ilgrosso commented on the issue:

    https://github.com/apache/incubator-ponymail/issues/59
  
    +1 to administratively hiding e-mails: are you saying that this issue can be just closed? Is the feature already documented?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail issue #59: New feature: allow admins to remove e-mails fr...

Posted by sebbASF <gi...@git.apache.org>.
Github user sebbASF commented on the issue:

    https://github.com/apache/incubator-ponymail/issues/59
  
    +1 to allowing mails to be 'hidden'


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

Posted by Humbedooh <gi...@git.apache.org>.
Github user Humbedooh commented on the pull request:

    https://github.com/apache/incubator-ponymail/issues/59
  
    Pierre, this has no relevance to the subject. How the ASF decides to utilize an implementation of this project is their business. It has no impact on how we decide to implement new features.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail issue #59: New feature: allow admins to remove e-mails fr...

Posted by Humbedooh <gi...@git.apache.org>.
Github user Humbedooh commented on the issue:

    https://github.com/apache/incubator-ponymail/issues/59
  
    Emails can be hidden by deleting the digest. This is what edit-lists.py does currently. The original email is retained in the source index, but the digest is removed, thus making the email not show up on lists.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail issue #59: New feature: allow admins to remove e-mails fr...

Posted by Humbedooh <gi...@git.apache.org>.
Github user Humbedooh commented on the issue:

    https://github.com/apache/incubator-ponymail/issues/59
  
    no, the issue can't be closed just yet. You can't delete emails by ID, only mass-remove lists with that script.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail issue #59: New feature: allow admins to remove e-mails fr...

Posted by ShaneCurcuru <gi...@git.apache.org>.
Github user ShaneCurcuru commented on the issue:

    https://github.com/apache/incubator-ponymail/issues/59
  
    We need to clarify documentation, to be specific as to what happens to the data and the user access thereto after using this (new) feature.
    
    That is: If an "admin" uses this feather, er, feature to delete the digest of a single message, then:
    
    - Users can no longer see it in list views.
    - Users will see - what? - if they hit a URL they've previously saved that pointed at the message?
    - If the message is in the middle of a thread, how does it affect threaded display?
    - Does this actually delete the >quoted content from other emails in the thread? (I'd presume not, meaning we need to clearly document that, lest someone think this is a secure feature to scrub data)
    - What happens if someone else wants to mirror the ponymail dataset?  I.e. after you use this feature, and I run a script to suck the config from the server to my local machine, will I get all data, or will this message be missing from my end?
    - Does this survive server restarts/reconfigs?
    
    Apologies if some of those are naive questions, but if we're gonna add something that makes people *think* they deleted a message, we need to be clear exactly how it works.
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

Posted by Humbedooh <gi...@git.apache.org>.
Github user Humbedooh commented on the pull request:

    https://github.com/apache/incubator-ponymail/issues/59
  
    step 1: how would you define an admin? :)
    It could be done, but would require some rewrite of the elastic library to support DELETE operations


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

Posted by ilgrosso <gi...@git.apache.org>.
Github user ilgrosso commented on the pull request:

    https://github.com/apache/incubator-ponymail/issues/59
  
    @Humbedooh maybe this is not necessarily a feature to expose via web UI, it could very well be something accessible via CLI (as other tools including importer): in this way you can safely jump the "step 1" above :-)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

Posted by Humbedooh <gi...@git.apache.org>.
Github user Humbedooh commented on the pull request:

    https://github.com/apache/incubator-ponymail/issues/59
  
    edit-list.py could be enhanced to do this, I suppose.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-ponymail pull request: New feature: allow admins to remove e-mails...

Posted by PierreSmits <gi...@git.apache.org>.
Github user PierreSmits commented on the pull request:

    https://github.com/apache/incubator-ponymail/issues/59
  
    How will we ensure that this is inline with ASF policies and procedures regarding mailing lists and postings?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---