-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Check pad deltas - Ready for Review / Merge @muxator #3717
Conversation
From ether#3717 (comment) > Afaik I used async / await that's pretty much all, I think I had to do some > polish because something was broken, remember stuff like pad.getPadAuthors was > b0rked in 1.7 or so
Mheh I didn't rebase but I'm pretty sure the showChat stuff is good to merge so all good to just use this. |
Updating
So this PR just need to add |
c9e81ca
to
0d3e825
Compare
So I went ahead and downloaded Marcel's original version from the url you mentioned, and then put yours on top. This results in the following diff, which is difficult to understand: 0d3e825 Could you explain it? From the initial comment i thought it was similar or equal to the file you linked, but there are many differences. Any pointers? Thanks |
Afaik I used async / await that's pretty much all, I think I had to do some polish because something was broken, remember stuff like pad.getPadAuthors was b0rked in 1.7 or so |
Committed by muxator on 2020-03-22, taking the raw version from: https://gist.githubusercontent.com/marcelklehr/a78d293571e7f06e3cf9/raw/c65b496402a752fbf746ab7d3736381b9b084da4/checkPadDeltas.js
From ether#3717 (comment) > Afaik I used async / await that's pretty much all, I think I had to do some > polish because something was broken, remember stuff like pad.getPadAuthors was > b0rked in 1.7 or so
From ether#3717 (comment) > Afaik I used async / await that's pretty much all, I think I had to do some > polish because something was broken, remember stuff like pad.getPadAuthors was > b0rked in 1.7 or so Fixes ether#2107.
564297b
to
d2aa265
Compare
Pulled in, thanks. |
As per https://gist.github.com/marcelklehr/a78d293571e7f06e3cf9
Requested in #2107