SessionList title bug

[maguireong]

No description provided.

[codecov]

Codecov Report

Merging #138 into master will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #138      +/-   ##
============================================
- Coverage     67.45%   67.33%   -0.12%     
  Complexity      672      672              
============================================
  Files           105      105              
  Lines          2338     2342       +4     
  Branches        267      268       +1     
============================================
  Hits           1577     1577              
- Misses          664      668       +4     
  Partials         97       97              

Impacted Files	Coverage Δ	Complexity Δ
src/main/java/seedu/address/ui/MainWindow.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
src/main/java/seedu/address/ui/RightSideBar.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 295a11a...595797f. Read the comment docs.

[kelvinvin]

Still updates the title regardless of the command
e.g. cadd n/Johnny p/12112 e/awpind@gmail.com a/tekong.

I think we can check whether commandResult == CommandResult(MESSAGE_SHOW_SESSIONS_SUCCESS) before updating the title

[tanweijie123]

Looks good 👍

[tanweijie123]

Try to remove debug statement before merging
System.out.println(commandResult.getFeedbackToUser());