Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Public delete #455

Merged
merged 16 commits into from
Sep 2, 2019
Merged

Public delete #455

merged 16 commits into from
Sep 2, 2019

Conversation

neuroscr
Copy link

sorry in a rush

Beaudan Brown and others added 16 commits August 29, 2019 17:11
Fix database helpers
6392c77
Add removeMessage function to conversation
5a70d88
Add background event listener
3332cb2
Add deletion polling
7d0b300
@neuroscr
rename deletePublicMessage => deleteLocalPublicMessage
d5e6ffb
@neuroscr
remove unneeded $ on badge
807b32f
@neuroscr
use newer delete system every 5s, deleteMessage(), store token in ser…
33572ca 
…verAPI, serverRequest refactor, registerChannel now returns the channel found/created, change baseChannelUrl base
Fix db stuff and remove unneeded message function
1b4105a
A prepare for message deletion
e4ecc5b
Only show delete option if have mod status and show different modal f…
351fa09 
…or mod deletion
All the API updates, enable mod status retrieval and allow for messag…
8d77d6f 
…e deletion
@neuroscr
Merge branch 'deletion-final' of https://github.com/BeaudanBrown/loki…
95cca85 
…-messenger into public-delete
@neuroscr
allow yourself to request deletion of messages
3178d67
@neuroscr
replace publicSendData with LokiPublicChannelAPI instance
133f2a5
@neuroscr
use LokiPublicChannelAPI
dfd6039
@neuroscr
sendMessage(), getOrRefreshServerToken supports forceRefresh, make mo…
56a4a31 
…dStatus a property, additional comments
BeaudanBrown
BeaudanBrown approved these changes Sep 2, 2019
View reviewed changes
Copy link

@BeaudanBrown BeaudanBrown left a comment

Choose a reason for hiding this comment

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

Some small style/best practices modifications, but I will make in another PR so that we can merge this and get some testing in before release today given it is still American Sunday

@BeaudanBrown BeaudanBrown merged commit f4e76f0 into oxen-io:public-chat Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BeaudanBrown BeaudanBrown BeaudanBrown approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@neuroscr @BeaudanBrown