From 2922156e45fd68f40801cc2bf6dc8563b83b5c3e Mon Sep 17 00:00:00 2001 From: FionaQY Date: Wed, 23 Oct 2024 17:36:46 +0800 Subject: [PATCH 1/4] Fix clear bug and update docs --- docs/UserGuide.md | 29 +++++++++++++++--- docs/diagrams/DeleteActivityDiagram.puml | 20 ++++++++++++ docs/images/DeleteActivityDiagram.png | Bin 0 -> 23621 bytes .../address/logic/commands/ClearCommand.java | 2 +- .../address/logic/commands/DeleteCommand.java | 2 +- 5 files changed, 47 insertions(+), 6 deletions(-) create mode 100644 docs/diagrams/DeleteActivityDiagram.puml create mode 100644 docs/images/DeleteActivityDiagram.png diff --git a/docs/UserGuide.md b/docs/UserGuide.md index 2446aecca90..554e12bb05c 100644 --- a/docs/UserGuide.md +++ b/docs/UserGuide.md @@ -439,21 +439,33 @@ delete ``` * Mandatory Field: `` * Note: The provided `` must be **greater than 0 and less than the total number of customers in the list**. +* After entering the command, you will be asked for confirmation (y/yes) before deletion occurs. For detailed explanations of each flag and acceptable arguments, refer to Sections [4.3 Flags](#43-flags) and [4.4 Arguments](#44-arguments) + + **Examples:** - Remove a customer with a specific index (e.g. at index 12): ``` delete 12 ``` + Confirmation prompt: + ``` + This will permanently delete this contact. Are you sure you want to execute this command? (y/n) + ``` **What to Expect:** -- **On Success:** +- **On Success (after confirming with y/yes):** - Message: ``` Customer has been deleted. ``` +- **On Cancellation (if confirmation is declined):** + - Message: + ``` + Command has been cancelled. + ``` - **On Error:** - Invalid index error message: ``` @@ -474,19 +486,28 @@ For detailed explanations of each flag and acceptable arguments, refer to Sectio ``` clear ``` +Confirmation prompt: +``` +This will permanently clear all contacts. Are you sure you want to execute this command? (y/n) +``` **What to Expect:** -- **On Success:** +- **On Success (after confirming with y/yes):** - Message: ``` Address book has been cleared! ``` The application will remove all client data from the list, effectively resetting the client database. +- **On Cancellation (if confirmation is declined):** + - Message: + ``` + Command has been cancelled. + ``` - **On Error:** - This command does not typically produce errors but will have no effect if there are no clients in the database to clear. > ⛔ **Danger:** -> The `clear` command is **irreversible** and does not provide a confirmation message before clearing all records. Once executed, all client data is **permanently deleted**. +> The `clear` command is **irreversible**. Once executed, all client data is **permanently deleted**. > > It is highly recommended to **avoid using this command** unless absolutely necessary. @@ -678,7 +699,7 @@ The data in AgentAssist is automatically saved as a [JSON](https://developer.moz Install the app in the other computer and overwrite the empty data file it creates with the file that contains the data of your previous AddressBook home folder. ### How do I change the remarks or credit card tier of an existing customer? -Use the [`edit` command](#feature-4-edit-the-existing-customer), and specify the `t/` flag for the credit card tier, and `rn/` or `ra/` for remarks. +Use the [`edit` command](#feature-4-edit-the-existing-customer), and specify the `t/` flag for the credit card tier, and `rn/` or `ra/` for remarks. If you wish to remove the assigned tier of a contact, simply use the `t/` flag without indicating a tier. ### Why am I getting an error when trying to edit the remark of an existing customer? Ensure that the command syntax is correct, and note that the `rn/` and `ra/` flags cannot be used together. The `rn/` flag replaces the existing remark, while `ra/` appends to the current remark. diff --git a/docs/diagrams/DeleteActivityDiagram.puml b/docs/diagrams/DeleteActivityDiagram.puml new file mode 100644 index 00000000000..9238959e296 --- /dev/null +++ b/docs/diagrams/DeleteActivityDiagram.puml @@ -0,0 +1,20 @@ +@startuml +title Delete Activity Diagram +start +:Delete command is given; +if () then ([index within range]) + :User is prompted + for confirmation; + if () then ([User confirms]) + :Delete contact + at specified index; + else ([else]) + :Show command + cancelled message; + endif + else ([else]) + :Show error message; + endif + +stop +@enduml \ No newline at end of file diff --git a/docs/images/DeleteActivityDiagram.png b/docs/images/DeleteActivityDiagram.png new file mode 100644 index 0000000000000000000000000000000000000000..9c2b3bcdf8dd59fd615e95c62f727681bc9f138f GIT binary patch literal 23621 zcmb@uby$@9_XR4Abc2L+cZh(rw1jjwI!J>kFdz*Qil7K6ID|;2w3IZ8G9Z#7jZ#WT z3wZZ<&iVfSz0Y&+d7g7TzVptT`?L4jYpp%;*Y!0?i5Q8_ojXUWt)*sk?i|)f__v-A z8@4>|)X{=J1pL)4{GGjhf;?{A^gpM0!|R5hqyG&THm4vqSATz>+j63!J|2!<{&ze* zM4Y`n?+(6ag+ut>GO_UgZ~HkcI8I=eXUc$nml%cm!C8W3ePSLRyH#hbCYE85@@}9Xrk9mir6RLW_VDRNq&)d zvQcTgnYB#9r>a7_^D38F&Ni7=!ha@S@+9)oy99>FWo)H4`2O69ak)QAA_JZ})SKRB zP3@P(L&UUvvf3l}{nAhwA3Q-9DAny`EW2jLW|m>3UnF9E{MzOm&F=S=! z8=&=LUp$k`cEmenClPtv!FPegabN8I{u_6>kv>^9`};KPFZm)A*Y7&nN>)FNX?PyP z%(G?0ZqRE$fc3L6cW#nB@tIJe65e-2E0dc&$r^5&v1IS`*eh4fHOU2zh@O+S%M$40 z#5x`wspO`;_oSI>FP?d071a(#kh4#pJ7;C7t)^lUX!qkqKnRWLm)IziRhGxsv9-f- zx%^d$+0;`wa^4o^@DK-JZg3zxgZY z;>P6j4X=r>O_fSJnXK)BRktZMhXqZjAoUyYq{Bj^&bhahPcxH}awA8%MP~zr?z~%R}L7Ch>If?Y5C8Y}^ z!dA6fPt?Ed|M;>Hm8}$Jm#-Bg6KNHfrW!-^5ze^1*vIjRllN+z?TY2*RgzGx+nY13 zrDo;FTV3RmF8$%D=JxkY2i+2)Q1`Zb=uYiYHv!>aiovSf{ye3x7Z ztBDoX?-8Q)QSz307Zi}S6l$uf*k+Uy`x}!rF7F;U-Tv-1l+yucQImdGQ)rMKYe_`I zcRgP#MgN5ye?nyUd1{`e{uG=^B6<;02gjUC;JKu(Bm`d2fCE>Ll$02Ten4eVo$dxv$LuJq4xa zmCxySdYQZ(99M>N*K)tqxtllN^;|`TuT8RsofziNd1Ol6icHOty`5k!zV^=HXejq8 zUsK4@j%CeFQX-<~3c>YOG}5Ex*7U(}0q4hgHzL=lGS$u}2i_-R*;#sr<6CLd^!m>F zm@7+a)CB8yQ9rM*Z)j!h!%p{JT}#i)io_$<(9~R1-8uWc`ErVrlQTs{fn!?)UQyR{ zjb;)v<$H>#KS$rQ`y^h+pCfgnP8de+kCj_9ytD9}X$ewO!=;-E`K6cNMR<00W>Id_ zBw=)RxJd3U`>`qPPw3gn(aXF%hu#MlBssoAT=einM@K|NXhqmW@5Pc_@SA&!MTi$K z!||S`&A4z)=KDfVqVt9M627IyM@qeG5WHor-VS#Csr<%O38kHI%`vUXk)v(*exKb= zGY~rv%22%bMvI1?_s6}c!^1;}GnKl=*c*dc7gh<#FG?}EQ(^bGDJ#Fdve^<ncjpm@}@=;%y>xc^YIlYQ#Q8@nM-R@{y?bK71pcYk((0srUaU*cV z;Q@6Fo*KnLC$Z?^t)EO2{rnNiB-Ri*aW}Ls46t0eOOpwYB+8Uw3Bqs6_Cw)`Ro%Va zVNDAW>-w^ez{(u8cq-FUvr8FYv!4<*`7I|@olMoaR}&pM?M*k`USr3lQz)G|IQ-eX zNm9W3Yxd8XIVn2xl6&l~f#MPF05z}vuB4rkT80v0kXcC*ohD*?e7L>1;CBC7`b6}= zIUeRvgAif)O!lj>qHTxRvXm0PFMP+1o4o8by@DvaTg|yG_s`o`*4?K}a#Z|@kg~eU z3727{m%AtQttPa$PqBX_M+HZTQ`P!0!gDFr^>Y!Wbh$O68@(@@>Jb&oD>eXLwi$D-cjS@tu8(w~!GVPpq3?~Y+L)Q=_iErE~E@Rc9JVy#h z)5O!qZ#Mg{KJNHXBkkt~HQ+%YGcWrGzHj|{&!tlKCCu~9Jag{;n6l^zVa^qQz~9z? zW4s13-o9jS#=YPBJ)IDapY;Uj6s~wqN#)D``XCv0c2wB0>moMN=Qo_Ee&x=ZNn1!z zJFn?ySldb;^Jnr9qmQDLY10Msqwn88SQ)-LZ7_ta*LcXtQMgq`Ar-p`Y@#L!Yw^I`9-sOjwlNM59=8eVgz8P|I7g{G}TD=QJQ!Oi3@F z-|^>U*L`w&ej{9dLPPZgq{JguB|7nqgW0f0yW@7jA2;7UTwESVcfMflE0K_=rlE0c zdElX{8ksHrf&MAKSIp+u=5b~8u@saN_E?cxY-f)B59y-W+1p3)ffpafX_M?ho@qOn z35K(HkCu$W-$)8jtq|IQNE}Y(NE`w(4@kg})SUwJg!vpGi;h>=)}$EY-FCxmS$_X? zgs0>{F6?-fS4L}kl8}&)LEH(KOT^17AxN$OEiNujK*lVV5r5u}&LHA#!)&M^6*u}h zy_48X8g0;=XbMLz-KW<#uDSHq5v5}(EjnKfaZAw*mT%OBK-1L7-9Otcv-Fdko02-xTDhcnrY|pE8Wy zvqf!Nk#!IX6I?sWo*9~)tpS@e7C%0{O6RQHJqQCfJ zeVXxd^JZID0TdQjYAg|atw%{oy*m}Q&0>~j2F(W9iX7zMz79Qk(|qW^y}j+m&JoyM zNP%7^?X}Lz%Oa*1u@LIsT1t(Kih5OfySONZ(C0#tRh@vyxwx27`Kc$CO@7>iSf#nS zxscfCywt_q#bRq43FqB*-wRgGb;kN{x9AI~Y-RsB(X{dNdDtCuo%yu)A(Q)9S%>P! z=aByh{=R9lFVV~C^m%69#1MbF^Z-eUUI1~24poSjAhCMb1$@z!e4 zB!W*uR@U;QO&L!`u!4F@^h*I&F-3R1&j_S`-P^KwYL$xT)OwsOnaRf4eU<1I{QMea(IX*}u#m~#eLoiwPwuz7V)$X<*YZFiss2|24olojT0t|K1{ojM ztnjS(6$WJ5A=?ukJ-6Y!o-)gtCDNZ*hPTs2Y)edvjmoX+F3*M-hwwh2p#A>@$(HN^ zEHt@|mY@}X3yFw$L2;n_3DGjx3fFcA zS4Rq=WOMu;O$Df>;~me{?Ym-`=T3G}o`I5*-F=iQB_#zhn3ZzC+l>Vu*jL!KTGiat zvn(3BHQiYK^`L;-i42Z&4Kn79r&G>=JN)>gC!$c?a?9Vd!^p_U(iGety}mKPL6kPY z(ap0fG|cbZm-Li-jbR~dh-yIwvq7|FE7Ie7U*LLq!_sTQ>hlWM{B>f8XkDV@G~mJg zyJ@#Sih$hZo}4b^&ZUKa_99xeeC#z;qgL2*xRCf-g=G-~0RKD9V(-Hzq-zWieyRU$ zYCk)6XKYiluCS49TM5snL#C~yL)Nh0FH(REEtswvr+~Rfif3abrq|4h?>v|GL~hO~ zG|{MTsGzBAZ^PZae=0bjTBr77s_tmY`>{0a9x#Yf_)=ymX<1rO>pgo0k-3A4mcduJ z7B@fqQuA86NZn!mW}2{d=;_}04q0?QJ~AyiOvR%)^z=ASJyFD_aVyii4KUK{tf=JA zSJ6})IOzA<(K1+Y01}03Hh|Lq-~QP@D=Ifsp@Ui`eiQcuKtBVWs9jcaa!!StLZZKi zsrh+r9Dx6^si~<|RaJVK(($IFHOU@OE35Wq%cAoy!iBZRCnbs4wH(HhGLaa_6Tid9 z!@J};WiqIvB>=s|QD%2{cX)XC+qcR^bST=Cvi9~rYon#h@MS8@UBx9N5RsGLcblJt z6C@`mLk1amMyRa-XZ=G#Pd{FJYa~x&&hq3G(E5_@KW7Zn($?lTF3^pq;*NGZJv}}0 z@?M~VUgeBiPVSBN6Q0f*fTHP_sUP$F-6$?zZB^&iSrdY-VS@*c=DeYyVX{zmI3+!F zJWi@MHq7w<ZFA1|}yY*6r}gb#Xtd}-)|KqS{$NPNI_ne3zR!MO#Z z6-%eYcxqm|mO#UcDc$r<{;N}P))Z0 z6CkA4;0AfR_|4Y{^u_=C01#Iy0mt^8!1cE*m>qY^$W&eqoQBJg!qFCXGHal*97aE4 zf#8w9ckC}ax|oajiaC=+=t*D@&2#9b7?e0KNVxPr;Wc=v@sNs&YDt&Ts=@2@c)fC< zSn)lSiZ6}6&iuY`WknU)VP}>)0m%UN@&PFNm0auSG5tx1SRvyCzhqp#l`>* z9vj;dDXf!W4o9F+TA|7sSN|N)4;=^YM4-6`7x(Z`_D4r$dk6@0MN3LByLHty>O31NoTynv>(?>l=_}t=dCRiws|~ zDY}{u2hyc|Nad&Q#jNUiGcBy=$}%>Ub_&s5-ydpSsMVxY2XN!QF;NXow{3$Li&268 z3nBy{&+`N^xpVYfs)-qeT~%UGE?+>(r(YS_v}S0`#QI$O=kioMdye&C#rF>1py9lYCKS0fc7haUNi~Mrc1!AjL(mZ zK`aN_VvDy;9sNabp=qam%3p6KZ@@HE^D`X9ZKfTCm<|2y8AwFz-Lj3((e3%A-gBy! z|7tABR!rWuhlhudP|HWcB|Lgg%8MSOB~SVKobYKsv>EbbRp_ZzO%@q)R<}5GV9PAO zjlk)2U%2-g150+!PJdUEtiCU=Uq!JQ~9w70L-=wp0L1I5TrOQ(~{7jY|ai<(VQebG|af%0XsHKiwi z0je%T1yAyZ#m&Ji*(ppaSG;!|x11e%eEtH(gj;K@sZiIjMN(qo2q5>#@Qh@Y>AJU{ z?e>I>>;mWULZqULtk5L~S0f`M=PF_KJJC*cpM@K@dP znh+{bSSa2tP(|{c?JN(dew4Xd_o>_P1KL=~Itq1%0Mea~n?mqXKPAZ?@Iy{bkk%J8 z%IQXHGWNfrv+a@0xM*M~jD)N|~)(ZY%d!O z^+DzKnrVqIU&x*=LVc}sAD2YSj{v~>1sO&mSel>ze0)Yrx{#$&;E)r(aB^U-Y7FGl z2P6yiZI|X$WbS#c&8oSX~E- z*>>?|o~Y|TUiw{3Y(W%L^yWST`!1!8m)-O10+&JHY)V!+R=3EV#R8K9mdnNuVY z`h=A65dMbk-eT@!LK$QiUf=w9+(E#~*Bc}ypE&0WNsvy`HA2^Yv}74^vgY+vz?8;W zc^sf4iH4XvoxSKts6DM>r$nB<5c6)ou#dfW7Yr?_%8;t^SHJ@Gw{rYyix2~z7 z!7)clNSGlX^xEm0o}wDn$J8JU;_|G^r*aX}hOSUN4)M;c4}JQ!{`>2=tjvwv|6 z=TpEjI#RAiMvtdIb&n3!aM8*mcU>XuG0V!cr_?cSJRMU9UeW87SQ?e)YsoK<*n~M|!Zh^OftXc0pD`IAfR(%bX zsmtsVFZ(UX>Py~vFJHn%w%u7b>!OAap)w=qW9XOu1*+}Zy1FEUgnVXY3?{_zahwkT zPw2@65Zqi(;n*ivO-xK&UHS5?0;T2UTVVC8sHphys2Y(gT>QD5G_nXn#i^CIoi1)OU+(*mr9qPHx8OxC*;7$d zmzorF$Pe&3beqDrGbm_j@!sv6XYgeQyvze}Jp9K7}WO6w*ndyQ9(w?ca&-tas!;9U%L>htp;2;o)&Sadb@FD1 zP07zihF4qq-GB)WD{QJ9I-mkVmCDA?L|sZRpz;#b>FL2N<-npY5cwhVop_1QNO#K& zuA-5wQ}d+4^|wZP9hgV8JjN@&f?B}@0I)qQj4KKHhS)a{vH%rvu(6{N&`|u|84z_o z079!0xEiepIlw5BKm4|%FD}BJl!?ULc{xI zy5xrY!f|rp*@U72?e;JLx|myvI#GgQN6YB|MY=K;mJG=xfIY!)zFNj4?V-E?h_b}I zGIV>P2O7;{)|y)*1&e);`cj#HS6DX;I()4e;}WdOweq~0F5aZwI*HgKr-* zfw!v7nEaQZbNpBXID)E+!ER-unv-trb8*y#)`@Ga-r+tHfT^)=wIcTb1+vw13<_M& zQ3i;Cbr!1?Nszh+|MnI)SYk9v;RQ zM45W%m^tXxO~r~;LOb1P!p4Qc8}f z{>Lv9O4c7h=s3~lN@07(wLynmf>4*u`P&pj4$KsHj@42-NqQ6nGFz(T?Qgo<;^fRS z4pmE*Jy`FEgi6f5P<*Q^In4)*z6Lz~w-3*Nw)GINi9qc^Pqx|Ri*7sJ|03GDOSKWc z{UBKJQEXB zVura(eIQRg+E{(Z0}2!6mBkQ>)A;6IV4Z#D-X_orLI`-h4ab5WGUJl_7_fdo4bBFA zXYoath3LMkN$7e^n^F4Dg@zh^p!F&5#LgRscDvu{CXT{7w6~a*PtGK*8)8rNk`b%8 z|J+wLDk+9+ywPIg@6QmZ4nlKpU)7WhZTM%{rjBzQ#hUWK;0Px`UVz1q>TsbxcjC9SdG$g zRg82@KLtH0Q|aEbJd3qa&JX6U7VY;KaDf1mkqW4ej@lCuZ5zFEsz4R{99*Efz8u%Z0#2D|Xx&BA;#x9x1>6sa6jVeWOm-RXy`mZA=jr_P^0 zzgAKo#ld1dYP!IsS_gUarHgG6su?aC&*5#lpt=5O<(wOV`lg!PojEHgX(LUy5s9L* zoP%s{s!Ocum^P>C1c8TdzSz@0;yxmNzycTI%mDzoAtHn_OBq@<*Tgh;+JJXRNjWo9e;1g(NblKT4k zLKahhh)Y1QxCt43SvNXR8iWkBaZ++}dIed?r7YHiOSdt{)zQ&`j$3^b7u3(T)0OZ; zE}*f92nnw@i+5Tp_`_?>T$!xlJ$|*41A=ktW$Fb|DX(dZ-Fcm-0!Js*Vpt06dO(vU zoTul&R>b18O9*6)^|ArcK06GX2c*ddkgRAgDH1%nOtGL4ck9_jR5%2gD5r=>1s||? zqMGugBVCxoxIe>WoV${@KIH=eroY9d0g@$ILiXbD#fYBc+JoAXeO1jr;SuOV`0YY>l)*LWT|L0{$f7*PeF!FRDYS95Lt z{A=EDq=sj(pM70K-(6?-4amNrMFTyfqpzQ4-Fb1h>^xM=3s>%LfC#-L$3qr1#iW4T z#W0`mxkd2q-GT0LqjdEHkVVeU&b~p-VSn1DxyEu4f4D_N$c%BC?6jYU<3Xn7G#B@! zWLSfx_f+sjK0TyhMj4-+#ZA9_#2-h174W{EmOhc`fOM0C)07uK@&T&P9YOQU+V&Op z?TRk_6a=Q%&}Vu^MxRcOek~_(=|`2O>$kBz^V*ngXRQ-3mkMH3u8VVHX^K!rGhS3c zMz24%iRo4+VU${#@4D~z>WNfDn_S?QLsP=Lk;SEVPoRfS`|^$iIacMj{Ju5#;QgMa z4er`qAcipeLfh-NZ)(6PExo_HI^t*iOcl)-OU^1X&g{#7^*-6Xlf!D8O|CuQg!`Xf zLrFs&VV1o4VA`Z$4mQ%0a1*Y9@A!z>#0qnPsF`Z$!E~K{m2%BJwPeyd&HVv z`mmAa^A<>KOC`L;E|6t6N>SQygV69Tz~yvy{`r-AsO zTh*OP>29KK%-%soM@N$@om|(y)bo>6$SED(?7LC7UH~}zo3mlt3+mNKKn`}T$QMds zGg9=v3cF>(8_0U-grJVEt-(@-C09C$x;q6Fb+xmm>?m>kqetj8;mj;IU>D`@DqL~y zeE@VxXACi-KRFkO%7ks+qtMV$(0FUIo;E&*I{c2bAOcmlI0?jlOmuWnDMp!lCUHaR z%g7+WJ2DnIu0kaKSE%J*>dUj-@Q~8?s(|G|purmBJXU6D0Z)oK&ngC`r@3NjEd#7g z8V(N5ucddAZlC#5fWt)zn3l-@-l%!Vcu5Yxzf;I(JBV~jP7N9X6IZ||Zn8NL)NB4< zTKH%iW#Aua>3YC{fUSD$U=<|z2I!uL(9VWtyUfXJ0>CE(5xuY#O(q0ag}f}_{E=%w z8x@5Bz{fOz0d8Yu&r(xe%6>0&fXV$$11=(iBgC$IuV~jTjr&GrZjZZvqb-D3-6e z9_XSAaOxF;;WY|bzJ}vzX7b7Z1}J3$xk@XA!_E8hrm*WS0R6b~D16cqFnd@)nC=p= zhOnjK0kNkMAZ$^U)84KW8>N+sd|CTEBO@*}Za^9GQK4QY&)sSTDW4$#vgC-k8aRpx z5Wr$`kWThM%;)bV)`r(;=nxoq2&SPPJ9taw*K@HoSf%4rfoOq_nUx-EM1uFuSzcuqdDD^YI z7uJf@#*ifd*I1DAvdbL^>lCjibzWbO_r~oT%u*8=1Sq&Y;s)>E80N3OQgENo5jql! zm<`B}d&}we987W*4Bj$wJIbf__x5O!Mr|QSLp9Y8vQ)R}hz<1hL%~tBK3duU?C>uQ zi5u^u2OE%kZvE!|>lhP^JqKmb*6PZn|J!;5?V=E_@C1-iKugEd47Z8V2yKBirb`q^ z6=R`2x*`f$Km!lTtJu?0BA0NnkIUV&2({l2VV=a8bPDMqc3^CL_qN3Ho z@U}Lmn_e=6As&hpoz3L|-4|y}vp7uoYGL8gGGIL?d+`rGNXW+xhJlb}DXzq+iPJP( zP}!uRJiy2kwYQ8@A`VDD5j3OlTv=-I9@GT=x~Wc$b~*yjMW-K-4Rte-Z-0y!D8&>G zAt-o3T|FAN59{GJdpyVF-7=i7P3?~#NWGkMAW+b1Xk`oTrliugL;NlRY|{-=Qe0f8 z09+IhX_Tug*?CkxDJ!$>N`qK;ulAFi=NRm{#mO6KUzb%>#J6klEB#kz6XdC0Goe4w zrO7Dlu^2fzI(F93DgBMWiM@?*EF7`b>*nH{eC8Ed7QM{!K%8e>4?Fl8kg!d8fl=u2 zeB+Ej?_hLd)0<|XPZKN+t@5os9x`1Hh3<@Yh85ct+v?nw2RTM}SD`uhz)K3p>KKI56~PRHy`NvL#oI0}Lo!pR36GZ+3D9bS zm9MOPt|AQ#a;oMOop}0r(5@(*Y~j3lIkA+3 z_(}}Hm4~}C0Kiss^2UhU`>I>b!YJtvxjDN94A09*14@;?2D0)f2mkg22GflCi zYb2sf%XaB-sl{k^z{!W-8#2A+G5Q#e0nL zRYnyyO=7tk_9{4uSpc45_=YW0n!uZ+b+q)vpzKb8wC7aZpi4C$t+%4q)>218EVwOR{}Lhs?c7^hYr-Nrdfn-j`$hdWT1q0T`U zEBQLU5WUhz*jg=m(=EgKQ_X6)CmNanNnoj6DQg+IIJ}4fh1u_v#R&@U)AbyxY3*8H z2dl)Wnw5|C&zaUMayW89a3J+ZPoc0S6r&E!M0-p&7uZm3$SpH1PCJZgUqh2Hx zwByqA{-1@TDPPLKOVj9)Cp&uOG-12pva7dVMVPWp_=#VMUK#Bx*Mj{pDvvrDpcPj+V@xcZp)JD@aziF#RI^4%fvG2aL5W)AnrW@z`$(6N_qbw%&!<_RGKLYmw zIACF6ZMOUxgtEV{aHQh@2J0O@KK>x|-Vh63_u9=t%bAvoUA0UX^Iy5HiF%j3y95GP zJ!|BJ70YInUIVW>-Ee5iP~GN%6)IMDu6)ggiWto4^AFZ>p9_v>P-&==zDs5 zIVhJatsC^kLl8U-y)qBmhde*V9T_W)F;nwuZHnY6pv zp(){M-G2x6AoB=2f1o3qLf;uLfYHjX=H|m`t(#ELQ+?ORgg;EqPl5xASw4^=0U9HW z1p;)u9q@83eRyWv9bT?yZ6tAMF4sg`^UuzKD^CS- zTaOB$RT5jf_D~jnF&P~?r9*xIm5ttF02upOe&qo7Z4U0^Gt)<|XH&d=xd_`HAftV6 zsi9L#GxDzh`n~GG)sV~$01##Rpj|G;sXL$@)VWNB;Je%0-;E3P*=<{@^XgUy5=`c< zD=luGe5QdLP>mK$y)Qu-m_QQ82duGhwNQ{*cgxxu+)pa+a$M8Rl#f_fSO_bk<;m{xr%b zc5NYG!ajqBsWZ!}`sXBg%{jh7{au3RM|v>b{r=~d);ppI97Ak)k-ICo z@<&3yQ6a{9dqN_g`+t26n9VhhzkFxr&7DNlo{N~mF@(|ZlfEEY21dWf1W{|4U;$2W zc`ASe^F;A7$~r6w$#biq5ld8C{h?+BC4+{6A&!de#&(#*Mq4u>%Jk*Ua>VFMSNl$` za*T<~g07P1Iao+g@82KLL3vcaXO6lC0wJzKDdz`!dV}c>_|kF-?{O?@lycYO7^JrINHfh4Kz$)* z>t5HpAO-l1Qs29ZUWSE0r`O3Z%bXsO86pJhI!5RE`_U(|#Ir;{ec zZz3qimv#+0JTU3Lw>-a%4;SkTG}!U)-}j@Vl`B_r4w)`qOiBe{{@=&(CSa~%V`Gz& zqG6$V44!n3T0sOyH|8btQqj|!gSj8XME{}weytN0V2Xg^!<-VO1q|Kw>(_^Q+3H*G z{jUFa^F3fN6t2mq0zCuCKPL*5#qGOrO-*o87M4_lw{Z<1%%Xv! z`pdd)f>MbZfO;{cg3eYF^=9iXh0C~~$M+rv2o@I?DbY@rGrDmH%4j6TY}lVu=t{uf zwoGb}2})t>ufZ!-s1|lT;2-Cl^`A+o&*Z3R?L#@SfNb#Cb`-oCFLm7m7_L`u)n&Pz z*BpF^k#><3Kr?`S=xF~71^_O6{~iFUMT%nBSxA27H$5dlO_>l5z+eC?a1JqnF@k9d z$U>*c%y{vlh<9M5*=y|PlAfoS8H6IfpeV|$yP49SOD5MQ6fUcNI;XI%qD=7D_71JV z{3I|(iMxgClVHBk|H17u3u|6ay72t=du$N;n_O57h2^bZ>CnpVolgoLbNDq)sLixn zM&LMJ6ITy+k`BueQy=vJi34%r*Aa;~p9Ay{H9xEg1}WF;KrICCu524LX+UM#o4?Nr zZ=PPjWCcp3#`_Y}Qm~0X0r&}3msQpemmiQoxk)kA&5tW|V2@C^yDlK}U0XY^`wQea>0VX>9kL|RsQ?;uwTw!cv#Fhw~*trD+N|Yn8LD%RG zv0tKu)R+g%8LVIGLFCq)3P6N!gD(>A8F~N)Akna%Z-mCXEB8O85h_3nm^OTdK%7uA zwctQh+O-a5D?Re}9fEW%WKqQiE~+6BG7|_(=vEkoUev3n^GkCJRux$Ny;23wUy|Zz zkL_tfU;MvUItecuGm`+usEhtlqu`0*_!O-&0}AZW&=6FksNtG#k9?Cs)%rlFHA)j* z3y~4SQ5|x)H4pYXc5)AB#47HSvvO41L+!tEw;OxI7`y}3>0>p7wya8_LZ5uEN58R9 zws|1=2i{86gI_e_b^cTM*F=n&lgr-bxtXG^%Bc5hS&Ef^-RA7pUa=%s-4jwZoLIxm zimYZlE^VN9jDUKsngaLKS;jmmru}EEAJ{z-jpcjCtAz#$C-*^2>dJ>pH#2TKUMmH{ zFmV~Uj}Ar9B_qlymJ-x(YrvcIP0E!_K>jz>!KmIE{E-p5Qmb@+vTJ4z6;`7)bn9xt zVPjr+pK7|7hiWcHuUS$sGs7^=cfi53%>k9b#X$R9-NC2!;XphF{g`_IT+HMz*uTK+ z5T}>WOw;W?hn|`o7`lQ2s)~F6!}SlaKlD^&8T9PJL|_GI>-daW?9V|McuJKy3K}xX=>^TofF*Kz z|A?Z@WAb&z#!GgH=Z7>5w}C}`ZuDm!jIks{M1bg+bikV}nk)fhW@QT$fUP0gq!>J7 zOjE)@0)!qwfNR%HSH&Et+uc6MZGUQ7VsY(+bz4Fmv2<3$YvRJ z(7@=+Y+H9pVZDiutaR)$m-|+{87EoZ-$OC>V2vI>a3O8`x{vL@Xqjc;54%B4EDLrIr{W}TqigiGzJVR0G)g3`A z%n|RkjmBMCv8;6Yp8`gkl2pD{a=>8%$;@yzxlDz%Ttf(QH8*g`z)Y>>m|9cvYxRwV z)gUEduq?-GsXwK7V%hPShZY(-?nAJXlX(SzJ-1U7*@1DMyDtD!*?ct)4L zoN}Y6I4o4d40^O?!n})WrBip??x$-zHy2WNVeRtqY1KYIB+uw>!0Z5(7EoSZuS1%_ z+ZMc9D=;oZBaU!&$>DMgfB_y(2*Lhl3)h@CdWfsb3tQ2cNy(k_84Z+B?V%+5gS<@v zeBA_)sV{xt2e7RAY1j1So2pIJN*^+atIo`c#}VoN`SuHv z)fI54=b-g6#Ndg{gJE)wDPKN;lu5c@8-?X2#Ls^VDiDb6`k-nFj*sS4=fr}l2KyaA zH@!0%$Z+wCo51RHh$4xCdm`Y(2P{|i;j#J=a1}2B(@UoODS*~(0^rya4q0Z4~HPbQFU*@MknsSETUn9+bfV~ zW&Tk)>}0zSa?(H_n-|W<`$c1>9}B5hET4fcO?pw@;-^jbt1S5-KQPJrjTT;%qU#iF z@?W@VGa7Onf%-c_wg2<0;DOMB8{ijUQiR7t1u!E>OzXvSkH3wOgW3o4WJ&SyDZgoa zaWJA5jNne6D|Ue@2n)q(JVtG7|D85_QQDEY*JA96!6C z_3>lmtwhe%M9GhZf-9EPpw)8*aP!JQ>-tZ51!QKBA>*4jjZd~1hX5KJDm+Z0I+ECzCjfE74SOPJ;=gHS1l3p9T(8x z8PXRSOa)Yqi;GJmVj}~E8#;fp-6;-m7IqZg?c4Oqx}|8Vf`0BWUIWha3^B(}$gpB& zz^S0%5dkgO{|x02Y{GiwF3Bry7?EI_R-Q^S1*!xjlKx_mo@WtmUqI_Z^=P6lxznYA zt>%7_S@k$a8h zU?H3iz>5o)$MFgjn+Zk+23Rf5i5IxQ@HOQ3Kb6JNN zhY4;Ba)rs;ORht6>Gq+9ACfKsmH-wiw}|nQ+Zkja``FLS4#R7(y zdYevfQ&sf-y$P^%f&=XuP;j;JdFpLY-5@p5@f)eN{_+7~3>fAmm@-(zVm$$U1;V-1 zxG-r#&UGjUq_f0j7zqm7>L75k>}k3657QG%v(b2KgeQfR+j$nFY!Rr=!_on|(iR%b zf2A$i$Me58+r2@Z1WX3m40QuhR2q33W9Ve0HAchiG_ZwUvu&r~+W9-T4n3ZZ2Z*9=hXt)c_`=|UfFK9~`da*x zw8w-GGoB=9C8m}$DUP#hg=)`;{&QARh=LV-nZ=}DF3K&8)9864T z(&k-VlG~CJTnTw!>iNnJl9+(<1P%iXKyrEmLY3h2K)L}4e(E$wZM8~@XU9+O!{msA zi9^TiEbFhQpdT-N%%p!pQhTL@Qsc?Rm(MQiK|+BQC1OP(&R?I$+%qAS5e68MU?4*q z_KgBnp##@7#2Cia8*aKL@$%Gm_3&a(50IfsZH!CgKhQ zWtS3H`NU#oIx(+N60Of~&`$V!d^ylRRiUKu#QE>4mdW~n?1kN2_A0=;I+_c(^A2lCXbghU>0Z2)0&iu!<`%6kw=(fN|m z5aT7%)J>S9e-iLHwn#+yn_g5ZG*_kI*~h^rxvrau{}9tbCET!aGDS}fhP&fb6o5MA zdIcH>Yi47?3@ffrH3lqy?_$4h+zyD!%h*}aM;$3N8J6ZHx#{XtifhLZA2?&RHcpMwc%;UF5;=y;R6zZ<27D=3 z`(lhJ^ZzO-*Fg9Ycyp=uEvz3lJ$9QPMKEt5>y!*`N=LaSxDj`axn>xuQcLwfsob`> z0Zki2#n5rJb6-v@2&z|qW}uxG9;LK^@iFTiBKux}#f3Xl&MdNi-RQtXhyNQjaFP%6 zkAVcXpiD6YG<9;EtdOd#_<=t)gyL3hZj5rQ?rHyX7VH9ju7EzqM!&l;5`((b@y~>UFU*}%WW70m%J9pWqzTLfar|EZ z2#gxu18WY3Aufk*na<#*;^ABrFpP=cr(iIlBO^l4&;b}?;44xh*{nijX*ippFHwyH zEp|9bj&ez$pidQF?dl=nz2n`E$iQ&29%sLYJBH5#W$Iy)OFn34?$bispXtR9%5^_1 zSIB8&9&uhgPW*yBMMsIErH+IVRE#Vo7HKAP@3@u^m!AF3zAb1xzIqaUEw_Bc2cKUO z)amj^z$I{G*5868UhYOSZth!GpNVCz!;DS(8CERRpLlN-3y*D3tN>=|hMd6afJmT+ zNN7gY5l6By{mEN{!nH75^B%a$Iyy;o4qg<&87*2H*FrJ$q%jcRiT+2m7@C&ct;z@F zxJzgNa<#A68ESnM2miVD9t&AQ!|nJbQZwc0gyVE$madjt*Gsj0k37z`5;V+jeE)BUvyatQ8sg=MgEErWS0@XT{>3BUJAg8% zgI2urFCYN5SQ26dxnI2a?{U1hHvk*J*vkLZmNIap=O}jSGl_(Cq2LVE@>k&39=2!i z8t?V?fPp1$oGtLmpj3864nl9s1s%Hcj+a{}m0}>-!@+?K{|LsW|MlgJw50v(%SkOd zEqP*Vb2h+%z%StqG&}w2Rx*Q2M@NT0`o7pM?%%8ZG&fn>>L*4T$Eq23z0jI%W z5y!c=5qy$B1N@`c0JUnqiSIzh^micfCyYSGdwVT-14to;Uvq(01hlH>!E{L)vI_{x zVq_QUD3KYnU}!)U5$HM0z0Q8u{?Qx+>bm7Gx!OZJ6)B=`1pUb{9kgV=4i0|&r}2z} zwr^?=3rd3atai(SKJxXB-^?qW)2-rXV|zPYbJsq>wK_XTcn!?xXV=Rt#E02G8ViNG z+0^(d%4la9xVUh+y`5tB_%A+SS`$3xKwrUNQ7YU#h%fv%E16AH=+4+ zP-D>wG(+`G7!!;mCf|r18GOv0*o3hD63n016B>yc0v`!v(#0K~&)z7fZDUUC@FajV z;&;|QLqI0{stzP%lKBU}L(anUafNp8>{s?5zMGYuMjicXA)gEMm~eUAIAUv$y>3P8 zb12?}RyhaWYQ4%VgUQPALY>5deR)w43J-q8AQ*<G-e2X?Ep_Zm@N@*)-!m_iKWDKRxnOOg@!*({GyLe zf_T5WN5wbuGLuSm^z(5N5qX&l8Rj%Mw%b(PgGJS)HhHyR0;fKBYL6!PAtL3y!8Z2$M6gSda00Moy z>6ia5-72Iqr``}o3ZoI?b(^1ypS*eL^a}YA^J*0ToZNQ3c6uJX?1Lg=NdFRMSC@4JEA= zEy;$ON6*%#LkAV*dH-fRp1+>gvwxYF`OR;B^ZR{|&*yz54+%7<>sq=MrcA|`?Pae! z07J1vR4q}x{a(WN-={Ke64`$aMTd)T)w3%9>W$j~AF5A@TfB*<*cOTR70)Prb4mj% z{;%-Va$~G;t-13my6lY!%pg+dJO5P~)X!a&xty8Wf$T_mL$i%n)K)j3mE}D)k>JCM ztbO-Zj@31ot_@qa@pMN)6~Wj<$O$YhomNY=MUr-HKY|IGrxb-%ELvXUp z90Yg?t&Bv&>fsc?l~}0D3P0KvP)?{A1caA1 zrlwQ=Yi4m%{mSg`T#Z{lo^H@G#-JdJWOm3&VdalA$J@B_x`$q~?YiC$&>>Vi>?mcp zCY8|@sZ~Ak8~dqj>V=SrNLB23@16T;>zADRvt@60MQg5KXk&^Qd*d=g+5ScvMYd?> z@J-j{0j56{rE&P8>+0h&Z+*1yos%DGr@F_G*ixvPQ785U-#8grS>+rbLsYoze6*FD z#IuZKP8msZANC%6@Yimh0R#PTi^0Q1K-QI)G0N4RN1?{u7$dST;Cy8M2F;rn@3y7Q z1LfB30k2s3^!o`fns6Vtf$sNp8$V~ZojeczZ-N$Agn@guu|yAje&S!#^eValRNfPE zF8A6oelhCJtR0Qr2cPt0%f!3CdMuIRw;Ah?N?L+rkO!AL3A@9?5pN0}Uuc*eDQG~5 z=0cJwZ-W|N(M%L9x?-s#1QlE)lGK~5Cf~UP=yvVaidppOzKnjgE zlyB{&aD4@=!F1iE{R1feAt|_deW|%vuXW@ta?} z@dD?WUh8hf?2q?+Y4@*$pGbC2uq`!n-V{UfeNY zJxQ&1t^1Kcvs*;!!^P~Y19i4g*&;%mTCB)p=sVUEk9t>bg%IyvvZa9Zwutk?sln`E zA>6&DV)#g1`!U}qn)R+)PQ;g<(g8%W;YxDjzv#YTZ^--_7q1gJ60OuhkqrkfNfGBg z-T*!_BQp~wKsN5gOPY`JK8i^kgAZ*`3;_+_TDAkKipcv#b~~{omOGj9H(qz5LII9b5?wD3{|j|xVTB51;v1?kpf}sr6|bd z@dmOkt}WI#vHZ?#;V_mr;MTk_NWzor!evWFIh{f-;BI9)BO0X~&)zBHB7{^?g>{Zq z(`exJc9XKiz>{d32c%yIC!mYx5;9$POi@S2qIua-Q{)z!|CXAkS@6~n@07l&+n4@$ zCxek^d2^d9HUuSA)g&nL&{C{=RjQ=WC#Pj2|FG;l#HJa)fOKw%0y-3T<|O{9i(=Qx?+jb(*d1A9LJDz1Zyi@`s~h!(gc zNR8(#7!uqOIkLQZDCn%Q4No#QKIrWa81q}@=mSWYyd*RAA;4%iKf7S<;7DGbio%#c z%XV&$+SCq^&?*BjQY|G|Ps+ksw$n4A&JMT^ABGkqnnUr(Hh_|0F}<-maBX3-5kbf+ zo4^~CGXpR1p^hN_5z{FSPLUJI^0=WG3))(n$9M_{q)~w2DNeI(&&GbIPp>QbSrU|1 z-WOBR-xLl+zOOLs!A=o7+C+&Zn|~D@+*Wr>#P);*3#GpW>xthNeZg`&g#yyNs8U*W z;Ki@PgvVJRs%t*Ekc76R_o3ziHAACcGzyU`Lp_N@Jiq*>s+kFEq2pka$QLJ-Cn@Uf zVs0YE#I2t5PG=aj&u1r8waeU^2av81qho9{PjDt&Os*yo2oXwM4qR>X6=$NqUZX4q zxyP|-4Dx`p;h04hL6^v%2l;~O!`}AQkj)?504po_>h2ZUMqeZo#k8_OLjch!E(cHr z`)H0CM-|ky2l4fGXKvW!uVrFrm|d`4^y>vLqw_&QD%ObUT><|SSxhmJud#!9K{6DZ zL4)?*g6&#=9n4rmb=M5pIh8s2tMA@N{^m!1(!LxqYmTSyDc_fsk&%7bR^@6(2Z9Gk zUj}$cbE0QrWe2$gYN!Q0PEm*gLO)Hcc^I#986Ch%Jnc#4sTeHjRfN2P4d)fs(MI@o zx7V`gk_w}I>htQ#-_3IkFQc-4!G#mA#;<Jw19A&C)Ut5*A2hq~0BS^ym>_ zcga3TT;fT@i3^H~{LxUl(a7Z8I0{M@DJxxLW9bsYQBNlv_1U?(|HA7)yj}#lXAIsG zXcduSv_xH9s!w16zeED}XcAeUihQx>lVdH-nfZXmwIFB%0sxS>BgAy63zx3F!(#(@%6;#4Z&Er}>b3lg4)Zfq=5sq~|49dKXPNc}-Mu z+%+%+yqtDQZ(sS$bHaJoB-WWuZ48m6#1@5f0VUw)K;4P3l0a0!BZK$vhu*C_MNTi)nPsbPKPumhd!AHu{Ty2D{9X=k z`c3fu2zL%~O3OBfS;)wHHd9Xme&FmHE2R(_cTF90v*JJt(9;#0-Z3B-jl%6$Tsm$| z{`~GaD2*_n>erv-)eR4v?C9u#RDN~z=3Qxp!W=lc+%}1H4nT?D0l1=T@lw0M?&5(# zflXF>&;iHyfO4j}9;L3;d-%YZC?mU1gjsfinH zF_$K-{{u0_t>))L-6rNGmJ-vrb7FpU2wd{{sV*y|dI+;0{(e03N1f9@8$am_{iO_f P-V5Aayqp{8p?m)W9Y;u! literal 0 HcmV?d00001 diff --git a/src/main/java/seedu/address/logic/commands/ClearCommand.java b/src/main/java/seedu/address/logic/commands/ClearCommand.java index 637ee425c4a..4207a904dc9 100644 --- a/src/main/java/seedu/address/logic/commands/ClearCommand.java +++ b/src/main/java/seedu/address/logic/commands/ClearCommand.java @@ -29,7 +29,7 @@ public CommandResult execute(Model model, Boolean confirmationReceived) { if (confirmationReceived.equals(requiresConfirmation)) { return this.execute(model); } - return new CommandResult(MESSAGE_CLEAR_CONFIRMATION, false, false, true, null, false); + return new CommandResult(MESSAGE_CLEAR_CONFIRMATION, false, false, false, null, true); } } diff --git a/src/main/java/seedu/address/logic/commands/DeleteCommand.java b/src/main/java/seedu/address/logic/commands/DeleteCommand.java index 649883e3229..5014d97af5d 100644 --- a/src/main/java/seedu/address/logic/commands/DeleteCommand.java +++ b/src/main/java/seedu/address/logic/commands/DeleteCommand.java @@ -55,7 +55,7 @@ public CommandResult execute(Model model, Boolean confirmationReceived) throws C if (confirmationReceived.equals(requiresConfirmation)) { return this.execute(model); } - return new CommandResult(MESSAGE_DELETE_CONFIRMATION, false, false, true, null, false); + return new CommandResult(MESSAGE_DELETE_CONFIRMATION, false, false, false, null, true); } @Override From a8d65a200930af995f4fa3159958bf489f0fc0ba Mon Sep 17 00:00:00 2001 From: FionaQY Date: Wed, 23 Oct 2024 17:42:01 +0800 Subject: [PATCH 2/4] Improve tests --- src/test/java/seedu/address/logic/LogicManagerTest.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/test/java/seedu/address/logic/LogicManagerTest.java b/src/test/java/seedu/address/logic/LogicManagerTest.java index aa314c15249..22dae2824c2 100644 --- a/src/test/java/seedu/address/logic/LogicManagerTest.java +++ b/src/test/java/seedu/address/logic/LogicManagerTest.java @@ -1,6 +1,7 @@ package seedu.address.logic; import static org.junit.jupiter.api.Assertions.assertEquals; +import static seedu.address.logic.Messages.MESSAGE_INVALID_PERSON_DISPLAYED_INDEX; import static seedu.address.logic.Messages.MESSAGE_UNKNOWN_COMMAND; import static seedu.address.logic.commands.CommandTestUtil.ADDRESS_DESC_AMY; import static seedu.address.logic.commands.CommandTestUtil.EMAIL_DESC_AMY; @@ -73,15 +74,13 @@ public void execute_cancelDeleteCommand_success() throws CommandException, Parse assertCommandSuccess(deleteCommand[0], MESSAGE_DELETE_CONFIRMATION, expectedModel); } - /* @Test public void execute_commandExecutionError_throwsCommandException() throws CommandException, ParseException { String[] deleteCommand = {"delete 9", "y"}; - assertCommandSuccess(deleteCommand[0], MESSAGE_DELETE_CONFIRMATION, model); + logic.execute(deleteCommand[0]); assertThrows(CommandException.class, MESSAGE_INVALID_PERSON_DISPLAYED_INDEX, () -> logic.execute(deleteCommand[1])); } - */ @Test public void execute_validCommand_success() throws Exception { From f2c05abf7c64d606431cca0758c98d27ce6ead95 Mon Sep 17 00:00:00 2001 From: FionaQY Date: Wed, 23 Oct 2024 17:49:37 +0800 Subject: [PATCH 3/4] Improve DeveloperGuide --- docs/DeveloperGuide.md | 8 ++++++-- docs/UserGuide.md | 2 -- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/docs/DeveloperGuide.md b/docs/DeveloperGuide.md index 082cef4ff07..53b40ffe308 100644 --- a/docs/DeveloperGuide.md +++ b/docs/DeveloperGuide.md @@ -303,12 +303,16 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli **Use case: U1 - Delete a person** + + **MSS** 1. User requests to list persons. 2. AgentAssist shows a list of persons. -3. User requests to delete a specific person in the list. -4. AgentAssist deletes the person. +3. User requests to delete a specific person in the list. +4. AgentAssist asks user for confirmation. +5. User confirms. +6. AgentAssist deletes the person. Use case ends. diff --git a/docs/UserGuide.md b/docs/UserGuide.md index 554e12bb05c..530f35fca67 100644 --- a/docs/UserGuide.md +++ b/docs/UserGuide.md @@ -443,8 +443,6 @@ delete For detailed explanations of each flag and acceptable arguments, refer to Sections [4.3 Flags](#43-flags) and [4.4 Arguments](#44-arguments) - - **Examples:** - Remove a customer with a specific index (e.g. at index 12): ``` From e876e040389dd8bb82faa497ecd68bffd4e6b9b6 Mon Sep 17 00:00:00 2001 From: FionaQY Date: Wed, 23 Oct 2024 17:49:37 +0800 Subject: [PATCH 4/4] Fix Checkstyle --- docs/DeveloperGuide.md | 8 ++++++-- docs/UserGuide.md | 2 -- docs/diagrams/DeleteActivityDiagram.puml | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/docs/DeveloperGuide.md b/docs/DeveloperGuide.md index 082cef4ff07..53b40ffe308 100644 --- a/docs/DeveloperGuide.md +++ b/docs/DeveloperGuide.md @@ -303,12 +303,16 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli **Use case: U1 - Delete a person** + + **MSS** 1. User requests to list persons. 2. AgentAssist shows a list of persons. -3. User requests to delete a specific person in the list. -4. AgentAssist deletes the person. +3. User requests to delete a specific person in the list. +4. AgentAssist asks user for confirmation. +5. User confirms. +6. AgentAssist deletes the person. Use case ends. diff --git a/docs/UserGuide.md b/docs/UserGuide.md index 554e12bb05c..530f35fca67 100644 --- a/docs/UserGuide.md +++ b/docs/UserGuide.md @@ -443,8 +443,6 @@ delete For detailed explanations of each flag and acceptable arguments, refer to Sections [4.3 Flags](#43-flags) and [4.4 Arguments](#44-arguments) - - **Examples:** - Remove a customer with a specific index (e.g. at index 12): ``` diff --git a/docs/diagrams/DeleteActivityDiagram.puml b/docs/diagrams/DeleteActivityDiagram.puml index 9238959e296..90b6dcf6b36 100644 --- a/docs/diagrams/DeleteActivityDiagram.puml +++ b/docs/diagrams/DeleteActivityDiagram.puml @@ -17,4 +17,4 @@ if () then ([index within range]) endif stop -@enduml \ No newline at end of file +@enduml