Page MenuHomePhabricator
Differential D12043

[native] introduce RemoveTagButton
ClosedPublic
Actions

Authored by ginsu on May 15 2024, 1:31 AM.
Tags
None
Referenced Files
F5255038: D12043.diff
Sun, Apr 6, 7:53 AM
Unknown Object (File)
Fri, Mar 28, 8:43 AM
Unknown Object (File)
Mon, Mar 24, 2:32 PM
Unknown Object (File)
Sat, Mar 15, 4:47 AM
Unknown Object (File)
Wed, Mar 12, 10:00 AM
Unknown Object (File)
Sun, Mar 9, 7:32 AM
Unknown Object (File)
Mar 1 2025, 1:52 PM
Unknown Object (File)
Feb 27 2025, 4:38 AM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
tomek
inka
ashoat
Commits
rCOMMbdba9420dabf: [native] introduce RemoveTagButton
Summary

This diff introduces RemoveTagButton. RemoveTagButton is reponsible for dispatching the delete farcaster channel tag action whenever the user preses the "Remove tag" button

Depends on D12042

Test Plan

Please see the demo video below

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.May 15 2024, 1:31 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 15 2024, 1:31 AM
ginsu edited the summary of this revision. (Show Details)May 15 2024, 1:32 AM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: tomek, inka.
ginsu added a parent revision: D12042: [native] extend loading logic to include fetching farcaster channel info.
ginsu attached a referenced file: F1766102: Screen Recording 2024-05-15 at 4.27.02 AM.mov. (Show Details)
ginsu edited the test plan for this revision. (Show Details)
ginsu attached a referenced file: F1766107: Screen Recording 2024-05-15 at 4.14.13 AM.mov. (Show Details)
ginsu added a reviewer: ashoat.May 15 2024, 1:34 AM

There is some copy in this diff that needs review

native/community-settings/tag-farcaster-channel/remove-tag-button.react.js
72 ↗(On Diff #40203)

This copy needs to be reviewed

Harbormaster completed remote builds in B28929: Diff 40203.May 15 2024, 1:50 AM
ginsu requested review of this revision.May 15 2024, 1:50 AM
inka added a comment.May 15 2024, 2:27 AM

Can we add a confirmation popup? Actions performed by red buttons seem serious, and I think we should double check if the user actually wanted to do them. Like after pressing Disable to disable an invite link

ashoat accepted this revision.May 15 2024, 5:02 PM

I'm not as worried about deleting a Farcaster tag. When we delete eg. a thread, there is content lost that can't be recovered. Whereas in the case of deleting the Farcaster tag, it's easy to reestablish it. I personally think this is okay without a confirmation modal/alert

This revision is now accepted and ready to land.May 15 2024, 5:02 PM
ginsu mentioned this in D12060: [keyserver/lib/web] include communityStore in the initialReduxState.May 15 2024, 6:18 PM
ginsu added a child revision: D12060: [keyserver/lib/web] include communityStore in the initialReduxState.May 15 2024, 7:11 PM
ginsu updated this revision to Diff 41129.Jun 7 2024, 4:21 PM

rebase before landing

Harbormaster completed remote builds in B29544: Diff 41129.Jun 7 2024, 4:38 PM
Closed by commit rCOMMbdba9420dabf: [native] introduce RemoveTagButton (authored by ginsu). · Explain WhyJun 7 2024, 4:41 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMMbdba9420dabf: [native] introduce RemoveTagButton.

Revision Contents
Changeset List

PathSize
native/
community-settings/
tag-farcaster-channel/
109 lines
19 lines

Diff 41129

View Options

native/community-settings/tag-farcaster-channel/remove-tag-button.react.js

{ {
"name": "lib", "name": "lib",
"version": "0.0.1", "version": "0.0.1",
"type": "module", "type": "module",
"private": true, "private": true,
"license": "BSD-3-Clause", "license": "BSD-3-Clause",
"scripts": { "scripts": {
"clean": "rm -rf node_modules/", "clean": "rm -rf node_modules/",
"test": "jest" "test": "jest"
}, },
"devDependencies": { "devDependencies": {
"buffer": "^6.0.3",
"@babel/core": "^7.13.14", "@babel/core": "^7.13.14",
"@babel/plugin-proposal-class-properties": "^7.13.0", "@babel/plugin-proposal-class-properties": "^7.13.0",
"@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8", "@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8",
"@babel/plugin-proposal-object-rest-spread": "^7.13.8", "@babel/plugin-proposal-object-rest-spread": "^7.13.8",
"@babel/plugin-proposal-optional-chaining": "^7.13.12", "@babel/plugin-proposal-optional-chaining": "^7.13.12",
"@babel/plugin-transform-runtime": "^7.13.10", "@babel/plugin-transform-runtime": "^7.13.10",
"@babel/preset-env": "^7.13.12", "@babel/preset-env": "^7.13.12",
"@babel/preset-flow": "^7.13.13", "@babel/preset-flow": "^7.13.13",
"@babel/preset-react": "^7.13.13", "@babel/preset-react": "^7.13.13",
"@pmmmwh/react-refresh-webpack-plugin": "^0.5.10", "@pmmmwh/react-refresh-webpack-plugin": "^0.5.10",
"babel-jest": "^26.6.3", "babel-jest": "^26.6.3",
"babel-loader": "^9.1.2", "babel-loader": "^9.1.2",
"buffer": "^6.0.3",
"clean-webpack-plugin": "^4.0.0", "clean-webpack-plugin": "^4.0.0",
"css-loader": "^6.7.3", "css-loader": "^6.7.3",
"css-minimizer-webpack-plugin": "^4.2.2", "css-minimizer-webpack-plugin": "^4.2.2",
"flow-bin": "^0.182.0", "flow-bin": "^0.182.0",
"flow-typed": "^3.2.1", "flow-typed": "^3.2.1",
"mini-css-extract-plugin": "^2.7.2", "mini-css-extract-plugin": "^2.7.2",
"react-refresh": "^0.14.0", "react-refresh": "^0.14.0",
"style-loader": "^3.3.1", "style-loader": "^3.3.1",
Show All 14 Lines "dependencies": {
"lodash": "^4.17.21", "lodash": "^4.17.21",
"react": "18.1.0", "react": "18.1.0",
"react-icomoon": "^2.5.7", "react-icomoon": "^2.5.7",
"react-redux": "^7.1.1", "react-redux": "^7.1.1",
"redux-persist": "^6.0.0", "redux-persist": "^6.0.0",
"reselect": "^4.0.0", "reselect": "^4.0.0",
"reselect-map": "^1.0.5", "reselect-map": "^1.0.5",
"simple-markdown": "^0.7.2", "simple-markdown": "^0.7.2",
"siwe": "^1.1.6",
"string-hash": "^1.1.3", "string-hash": "^1.1.3",
"tcomb": "^3.2.29", "tcomb": "^3.2.29",
"siwe": "^1.1.6",
"tinycolor2": "^1.4.1", "tinycolor2": "^1.4.1",
"tokenize-text": "^1.1.3", "tokenize-text": "^1.1.3",
"util-inspect": "^0.1.8", "util-inspect": "^0.1.8",
"utils-copy-error": "^1.0.1", "utils-copy-error": "^1.0.1",
"uuid": "^3.4.0",
"wagmi": "^0.12.16" "wagmi": "^0.12.16"
}, },
"jest": { "jest": {
"transform": { "transform": {
"\\.js$": "babel-jest" "\\.js$": "babel-jest"
}, },
"transformIgnorePatterns": [ "transformIgnorePatterns": [
"/node_modules/(?!@babel/runtime)" "/node_modules/(?!@babel/runtime)"
] ]
} }
} }
View Options

native/community-settings/tag-farcaster-channel/tag-farcaster-channel.react.js

{ {
"name": "lib", "name": "lib",
"version": "0.0.1", "version": "0.0.1",
"type": "module", "type": "module",
"private": true, "private": true,
"license": "BSD-3-Clause", "license": "BSD-3-Clause",
"scripts": { "scripts": {
"clean": "rm -rf node_modules/", "clean": "rm -rf node_modules/",
"test": "jest" "test": "jest"
}, },
"devDependencies": { "devDependencies": {
"buffer": "^6.0.3",
"@babel/core": "^7.13.14", "@babel/core": "^7.13.14",
"@babel/plugin-proposal-class-properties": "^7.13.0", "@babel/plugin-proposal-class-properties": "^7.13.0",
"@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8", "@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8",
"@babel/plugin-proposal-object-rest-spread": "^7.13.8", "@babel/plugin-proposal-object-rest-spread": "^7.13.8",
"@babel/plugin-proposal-optional-chaining": "^7.13.12", "@babel/plugin-proposal-optional-chaining": "^7.13.12",
"@babel/plugin-transform-runtime": "^7.13.10", "@babel/plugin-transform-runtime": "^7.13.10",
"@babel/preset-env": "^7.13.12", "@babel/preset-env": "^7.13.12",
"@babel/preset-flow": "^7.13.13", "@babel/preset-flow": "^7.13.13",
"@babel/preset-react": "^7.13.13", "@babel/preset-react": "^7.13.13",
"@pmmmwh/react-refresh-webpack-plugin": "^0.5.10", "@pmmmwh/react-refresh-webpack-plugin": "^0.5.10",
"babel-jest": "^26.6.3", "babel-jest": "^26.6.3",
"babel-loader": "^9.1.2", "babel-loader": "^9.1.2",
"buffer": "^6.0.3",
"clean-webpack-plugin": "^4.0.0", "clean-webpack-plugin": "^4.0.0",
"css-loader": "^6.7.3", "css-loader": "^6.7.3",
"css-minimizer-webpack-plugin": "^4.2.2", "css-minimizer-webpack-plugin": "^4.2.2",
"flow-bin": "^0.182.0", "flow-bin": "^0.182.0",
"flow-typed": "^3.2.1", "flow-typed": "^3.2.1",
"mini-css-extract-plugin": "^2.7.2", "mini-css-extract-plugin": "^2.7.2",
"react-refresh": "^0.14.0", "react-refresh": "^0.14.0",
"style-loader": "^3.3.1", "style-loader": "^3.3.1",
Show All 14 Lines "dependencies": {
"lodash": "^4.17.21", "lodash": "^4.17.21",
"react": "18.1.0", "react": "18.1.0",
"react-icomoon": "^2.5.7", "react-icomoon": "^2.5.7",
"react-redux": "^7.1.1", "react-redux": "^7.1.1",
"redux-persist": "^6.0.0", "redux-persist": "^6.0.0",
"reselect": "^4.0.0", "reselect": "^4.0.0",
"reselect-map": "^1.0.5", "reselect-map": "^1.0.5",
"simple-markdown": "^0.7.2", "simple-markdown": "^0.7.2",
"siwe": "^1.1.6",
"string-hash": "^1.1.3", "string-hash": "^1.1.3",
"tcomb": "^3.2.29", "tcomb": "^3.2.29",
"siwe": "^1.1.6",
"tinycolor2": "^1.4.1", "tinycolor2": "^1.4.1",
"tokenize-text": "^1.1.3", "tokenize-text": "^1.1.3",
"util-inspect": "^0.1.8", "util-inspect": "^0.1.8",
"utils-copy-error": "^1.0.1", "utils-copy-error": "^1.0.1",
"uuid": "^3.4.0",
"wagmi": "^0.12.16" "wagmi": "^0.12.16"
}, },
"jest": { "jest": {
"transform": { "transform": {
"\\.js$": "babel-jest" "\\.js$": "babel-jest"
}, },
"transformIgnorePatterns": [ "transformIgnorePatterns": [
"/node_modules/(?!@babel/runtime)" "/node_modules/(?!@babel/runtime)"
] ]
} }
} }