Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

Update user docs #137

Merged
merged 19 commits into from
Nov 2, 2023
Merged

Update user docs #137

merged 19 commits into from
Nov 2, 2023

Conversation

Cloud7050
Copy link

@Cloud7050 Cloud7050 commented Nov 1, 2023

This PR works towards #127 and #96. It includes a full pass over the landing page & UG, and also edits some other docs.

It tries to account for changes/discussions in #139. It adds a TODO stub for the new filter feature.

  • After merging, check the generated docs to ensure there aren't formatting errors.

Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #137 (342d5ad) into master (f1604e9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #137   +/-   ##
=========================================
  Coverage     72.73%   72.73%           
  Complexity      418      418           
=========================================
  Files            75       75           
  Lines          1412     1412           
  Branches        137      137           
=========================================
  Hits           1027     1027           
  Misses          343      343           
  Partials         42       42           
Files Coverage Δ
...in/java/swe/context/logic/commands/AddCommand.java 100.00% <ø> (ø)
...swe/context/model/util/AlphabeticalComparator.java 71.42% <ø> (ø)
...ava/swe/context/model/util/SampleContactsUtil.java 25.00% <ø> (ø)

@Cloud7050 Cloud7050 marked this pull request as ready for review November 1, 2023 17:09
@Cloud7050 Cloud7050 added this to the v1.3b milestone Nov 2, 2023
@bwangpj bwangpj merged commit 5fb1b5f into AY2324S1-CS2103-W14-3:master Nov 2, 2023
5 checks passed
@Cloud7050
Copy link
Author

I've pushed some formatting fixes to master so I can see it rebuild live.

Looks like keys don't get special formatting.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants