From 00356ad54333783d22bbc9eb4988638807f33811 Mon Sep 17 00:00:00 2001 From: Harrison Healey Date: Fri, 11 Oct 2019 09:28:09 -0400 Subject: [PATCH] Revert accidentally merged changes from mark-as-unread branch (#3405) * Revert "Update mattermost-redux" This reverts commit 04b3897ecb2b4107d54f955525ddec0b9fa5f74e. * Revert "[MM-17068] Add set post as unread option to menu (#3202)" This reverts commit c7a29eea6a18f9fd918b62bfc843e52fdb56badd. --- app/reducers/views/channel.js | 5 - .../__snapshots__/post_options.test.js.snap | 150 +++--------------- app/screens/post_options/index.js | 3 - app/screens/post_options/post_option.js | 2 - app/screens/post_options/post_options.js | 55 +++---- app/screens/post_options/post_options.test.js | 2 - assets/base/i18n/en.json | 1 - assets/base/images/post_menu/bookmark.png | Bin 669 -> 0 bytes assets/base/images/post_menu/bookmark@2x.png | Bin 840 -> 0 bytes assets/base/images/post_menu/bookmark@3x.png | Bin 1023 -> 0 bytes package-lock.json | 10 +- package.json | 2 +- packager/moduleNames.js | 1 - 13 files changed, 52 insertions(+), 179 deletions(-) delete mode 100755 assets/base/images/post_menu/bookmark.png delete mode 100755 assets/base/images/post_menu/bookmark@2x.png delete mode 100755 assets/base/images/post_menu/bookmark@3x.png diff --git a/app/reducers/views/channel.js b/app/reducers/views/channel.js index f9c771fbb..3ffce2343 100644 --- a/app/reducers/views/channel.js +++ b/app/reducers/views/channel.js @@ -348,11 +348,6 @@ function lastChannelViewTime(state = {}, action) { return state; } - case ChannelTypes.POST_UNREAD_SUCCESS: { - const data = action.data; - return {...state, [data.channelId]: data.lastViewedAt}; - } - default: return state; } diff --git a/app/screens/post_options/__snapshots__/post_options.test.js.snap b/app/screens/post_options/__snapshots__/post_options.test.js.snap index 596d208c3..7139754cc 100644 --- a/app/screens/post_options/__snapshots__/post_options.test.js.snap +++ b/app/screens/post_options/__snapshots__/post_options.test.js.snap @@ -10,8 +10,8 @@ exports[`PostOptions should match snapshot, no option for system message to user > - - - { - const {post, isLandscape, theme} = this.props; - const {formatMessage} = this.context.intl; + getMyPostOptions = () => { + const actions = [ + this.getEditOption(), + this.getReplyOption(), + this.getFlagOption(), + this.getPinOption(), + this.getAddReactionOption(), + this.getCopyPermalink(), + this.getCopyText(), + this.getDeleteOption(), + ]; - if (!isSystemMessage(post)) { - return ( - - ); - } - return null; + return actions.filter((a) => a !== null); }; - getPostOptions = () => { + getOthersPostOptions = () => { const actions = [ this.getReplyOption(), - this.getAddReactionOption(), - this.getMarkAsUnreadOption(), - this.getCopyPermalink(), this.getFlagOption(), - this.getCopyText(), + this.getAddReactionOption(), this.getPinOption(), + this.getCopyPermalink(), + this.getCopyText(), this.getEditOption(), this.getDeleteOption(), ]; @@ -262,6 +254,12 @@ export default class PostOptions extends PureComponent { return actions.filter((a) => a !== null); }; + getPostOptions = () => { + const {isMyPost} = this.props; + + return isMyPost ? this.getMyPostOptions() : this.getOthersPostOptions(); + }; + handleAddReaction = () => { const {theme} = this.props; const {formatMessage} = this.context.intl; @@ -326,15 +324,6 @@ export default class PostOptions extends PureComponent { }); }; - handleMarkUnread = () => { - const {actions, post, currentUserId} = this.props; - - this.closeWithAnimation(); - requestAnimationFrame(() => { - actions.setUnreadPost(currentUserId, post.id); - }); - } - handlePostDelete = () => { const {formatMessage} = this.context.intl; const {actions, post} = this.props; diff --git a/app/screens/post_options/post_options.test.js b/app/screens/post_options/post_options.test.js index d2e7e9a0a..6be5ab8e1 100644 --- a/app/screens/post_options/post_options.test.js +++ b/app/screens/post_options/post_options.test.js @@ -26,7 +26,6 @@ describe('PostOptions', () => { removePost: jest.fn(), unflagPost: jest.fn(), unpinPost: jest.fn(), - setUnreadPost: jest.fn(), }; const post = { @@ -47,7 +46,6 @@ describe('PostOptions', () => { canEditUntil: -1, channelIsReadOnly: false, currentTeamUrl: 'http://localhost:8065/team-name', - currentUserId: 'user1', deviceHeight: 600, hasBeenDeleted: false, isFlagged: false, diff --git a/assets/base/i18n/en.json b/assets/base/i18n/en.json index a99e0da2e..057487627 100644 --- a/assets/base/i18n/en.json +++ b/assets/base/i18n/en.json @@ -360,7 +360,6 @@ "mobile.post_info.add_reaction": "Add Reaction", "mobile.post_info.copy_text": "Copy Text", "mobile.post_info.flag": "Flag", - "mobile.post_info.mark_unread": "Mark post as unread", "mobile.post_info.pin": "Pin to Channel", "mobile.post_info.reply": "Reply", "mobile.post_info.unflag": "Unflag", diff --git a/assets/base/images/post_menu/bookmark.png b/assets/base/images/post_menu/bookmark.png deleted file mode 100755 index df28ccf585023660d36c68fbc856b8e87798703f..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 669 zcmeAS@N?(olHy`uVBq!ia0vp^5+KaM0wlfaz7_*1mUKs7M+SzC{oH>NS%G|oWRD45bDP46hOx7_4S6Fo+k-*%fF5lweBoc6VW5Skx99p8++E9*{(|oHa#_AOTwPVt>)LW zjJSfBY<9etw|IPF@}^Dh)75^-`df!=OS;x$y1nSD!Gr*VZ^D|(R&~Ul-WJo4K7R|3 z^0!+H9(!H8s-wB<*F~!f%bz@z($0_KS^e&f`MQ{pync}tUlJ^~KXwT7OgI-{>-Z`B zjd9^^E~dieaZ$D{3uiwt&+_o(_z-wJ&#d;+;+MTImL;q4mj7MqwSNDX|H7tu zQi~Ex@{6n#g0oXgk~0)c&GifwG%Sn_EG#q?bQLm6N(zdt^!1CuvU)&`3g4VIy#pGi t0yC^6vm_^#K>-dt{e$&Alk@ZRL-TmdLD diff --git a/assets/base/images/post_menu/bookmark@2x.png b/assets/base/images/post_menu/bookmark@2x.png deleted file mode 100755 index 7f1db30044eff1ac0edf28f247dfb121135eecf5..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 840 zcmeAS@N?(olHy`uVBq!ia0vp^1|ZDA0wn)(8}a}tmUKs7M+SzC{oH>NS%G|oWRD45dJguM!v-tY$DUh!@P+6=(yLU`q0KcVS>y)vIg-4nJ$S8pEE{@e4Ktc8rPhVH|#~d;O+`4DBxh-K}VBF{F;usQf`0doQ+AN6@2kN!? zvQ^_(Z}G~FWLIx#E(+xP$0jabB=A&?ed&`WGBHz>PC34nf3SsJ^X8Us{5O(3Cr#CRAZI(GrT2g0w*hCom zNz3jAEo8`y`}#bHM=-HTwf0!sd(L??lUr9!(qziG^D17UH`Gcpb)L-WC!w>R%+ug( zF&FdMZSnN1CR0aY&xbQMicP;(Bp#cyU{Zddx%HQiM=ULE``lV2rt=g(^j%!#G1Vfs zu<2gNhrpvdUcR4@I=#pI{d=xt)-tvFPj@q1=lGaOzvcR_CZ6qQ{X6^WB7wDHYU1c{^%=>4QPF#5JNM zC9x#cD!C{XNHG{07+UHYSm+vAgczDx85&xdm}(msSQ!}X_gDf-g$NC~`6-!cmAEyW z6P4Kp)ZhrRp|~vFDmgzlH#M(>LBTyWFSRJKB)`Z?Avim=BsoLD)LhR{LBqn>z`{aP zL02K8q@tJHDlo%JGD~t&85H2a(?3|>GdVv`KQs^MkCfv5 S-YuX6&EVYAW?~aSX}0_jbnFY+*-<_W8NG zxjyz6%mfx5)93njVZVdRhgWMW1Ya+#dgZ$Ef#Vhpr3w+BjoaJ~>UiWR zU|aFZQ>dHGYqrT+-(#nP%yw^n{qEeGuv0eGC;iIQ{L{+r{oR;gAi=}ttfG_on&GBI z^3@-$9!?o^Jbx+i=Wg_wXj!Ui{tSy+B+;*4{3 z?+RHOw7lo#|31OQ{vzE{cA;aM1)oEwr`^tXN(W1iHGE@HU(#xG{*7zolOM^%?`gfTuOBDdtxb{aWivLMo1! zAO9)woMDCG(KYinO@ErYME#F+mHF?o4>FAl4w`0`@)eMa^sz6RI6 zbrqi<#%{VVveUzV#mj9gjLx1v`L0hzxS84HdT;ZDcYkvDQ@q7g>nsZ01$|w5i z;j|as@{Uew(|~E9S<&(z)6_>f;#))G9s|>)YKdz^NlIc#s#S7PDv)9@GBC8%HL%b% zvIsFWvobWaGBMRQFt9Q(*zd6flz9*ua`RI%(<*UmI43Hz4XD8pWJ7UTx>a(1Zftzq`@LI0`H;cW)z4*}Q$iB}!wQcy diff --git a/package-lock.json b/package-lock.json index 2020b0fa4..830ce926c 100644 --- a/package-lock.json +++ b/package-lock.json @@ -14133,8 +14133,8 @@ "integrity": "sha512-rUxjysqif/BZQH2yhd5Aaq7vXMSx9NdEsQcyA07uEzIvxgI7zIr33gGsh+RU0/XjmQpCW7RsVof1vlkvQVCK5A==" }, "mattermost-redux": { - "version": "github:mattermost/mattermost-redux#dc5f8c1f30cbc6a25d4304f2923f5b78e674099d", - "from": "github:mattermost/mattermost-redux#dc5f8c1f30cbc6a25d4304f2923f5b78e674099d", + "version": "github:mattermost/mattermost-redux#3bb89a5450b94da564d3a286fefb5945a1c3fca8", + "from": "github:mattermost/mattermost-redux#3bb89a5450b94da564d3a286fefb5945a1c3fca8", "requires": { "deep-equal": "1.0.1", "eslint-plugin-header": "3.0.0", @@ -14147,7 +14147,7 @@ "redux": "4.0.4", "redux-action-buffer": "1.2.0", "redux-batched-actions": "0.4.1", - "redux-offline": "git+https://github.com/enahum/redux-offline.git#885024de96b6ec73650c340c8928066585c413df", + "redux-offline": "git+https://github.com/enahum/redux-offline.git#4bd85e7e3b279a2b11fb4d587808d583d2b5e7b5", "redux-persist": "4.9.1", "redux-thunk": "2.3.0", "reselect": "4.0.0", @@ -17575,8 +17575,8 @@ } }, "redux-offline": { - "version": "git+https://github.com/enahum/redux-offline.git#885024de96b6ec73650c340c8928066585c413df", - "from": "git+https://github.com/enahum/redux-offline.git#885024de96b6ec73650c340c8928066585c413df", + "version": "git+https://github.com/enahum/redux-offline.git#4bd85e7e3b279a2b11fb4d587808d583d2b5e7b5", + "from": "git+https://github.com/enahum/redux-offline.git#4bd85e7e3b279a2b11fb4d587808d583d2b5e7b5", "requires": { "@react-native-community/netinfo": "^4.1.3", "redux-persist": "^4.5.0" diff --git a/package.json b/package.json index 7ef0ff97c..be2be758d 100644 --- a/package.json +++ b/package.json @@ -21,7 +21,7 @@ "intl": "1.2.5", "jail-monkey": "2.2.0", "jsc-android": "241213.2.0", - "mattermost-redux": "github:mattermost/mattermost-redux#dc5f8c1f30cbc6a25d4304f2923f5b78e674099d", + "mattermost-redux": "github:mattermost/mattermost-redux#3bb89a5450b94da564d3a286fefb5945a1c3fca8", "mime-db": "1.40.0", "moment-timezone": "0.5.25", "prop-types": "15.7.2", diff --git a/packager/moduleNames.js b/packager/moduleNames.js index a7c0e11a1..9a92fb941 100644 --- a/packager/moduleNames.js +++ b/packager/moduleNames.js @@ -303,7 +303,6 @@ module.exports = [ 'dist/assets/images/icons/word.png', 'dist/assets/images/post_header/flag.png', 'dist/assets/images/post_header/pin.png', - 'dist/assets/images/post_header/bookmark.png', 'dist/assets/images/profile.jpg', 'dist/assets/images/status/away.png', 'dist/assets/images/status/dnd.png',