Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Answer post UI #84

Draft
wants to merge 64 commits into
base: f24
Choose a base branch
from
Draft

Answer post UI #84

wants to merge 64 commits into from

Conversation

katcday
Copy link

@katcday katcday commented Oct 11, 2024

No description provided.

github-classroom bot and others added 30 commits September 9, 2024 17:37
@github-classroom
add deadline
aeb83cb
@philliparaujo
Test changing README to ensure branch working
62a393a
@heyanuja
added spaces to readme test change
c62ad7a
@KesterTan
reduce code depth
9fd93e1
@philliparaujo
Adding changes from project 1
56cce13
@philliparaujo
New custom theme, added search bar in category page
b348683
@philliparaujo
Added listeners to new search components
1698047
@philliparaujo
Renamed theme
3bc3ede
@philliparaujo
Removed old theme name
634b9a3
@philliparaujo
Updated README to use theme
a2dbab8
@philliparaujo
npm run lint
e20be27
@philliparaujo
Removing test README
b6385c3
@philliparaujo
Merge pull request #15 from CMU-313/search-bar
a81893e 
Created Slackers custom theme, added search bar component to it
@philliparaujo
Merging changes
db8f3f9
@philliparaujo
npm run lint
59843e2
@philliparaujo
Merge pull request #16 from CMU-313/phillip-project1
c0c83be 
Refactoring code to reduce level of nesting (Project 1)
@KesterTan
Merge pull request #14 from CMU-313/kester-project1
e690920 
Merge project 1 changes
@katcday
Adding answered field to post objects
f735317
@katcday
Adding API to toggle if a post is answered or not
57eec9c
@katcday
Adding unit testing for the answering API
cd7b505
@katcday
Fixing lint errors
53ac399
@katcday
Added project 1 Changes
5185f9d
@KesterTan
frontend and functional backend for verifying message
7826762
@KesterTan
changed theme to reflect post schema
95c1bd5
@KesterTan
added openapi documentation
653c107
@katcday
Adding meaningful message to console log in toggleAnswered event
5bca90a
@katcday
Merge pull request #18 from CMU-313/camille-project-1
e0e965d 
Refactoring `onMessage` function in `src/socket.io/index.js` to reduce cognitive complexity
@heyanuja
project 1 changes
8f79466
@katcday
Merge pull request #17 from CMU-313/answered-field
64d9e6f 
Add Ability to Toggle Posts as Answered Vs. Unanswered
@heyanuja
search algorithm to sort through categories
7512a1d
KesterTan and others added 27 commits September 24, 2024 20:58
@KesterTan
Merge pull request #20 from CMU-313/verify-message
0de0dac 
Instructors can verify message
@heyanuja
passing linter, removed trailing spaces etc
cd495fb
@yerimsong
Add or update the Azure App Service build and deployment workflow config
95ca18f
@heyanuja
reverting changes from search bar to experiment on test passing
1872b1a
@heyanuja
lint space fix
93c0479
@philliparaujo
Merge pull request #27 from CMU-313/searchalgfixes
e33ee67 
SearchAlg code with fixed Linter issues
@philliparaujo
Started adding post updating on search query
88a5ed9
@philliparaujo
Outputting topics without using query :)
9dce96b
@philliparaujo
Searching now works mostly
ebe4bb1
@philliparaujo
Final search for now
f70f7a6
@philliparaujo
npm run lint
d4a2d61
@heyanuja
Merge pull request #29 from CMU-313/search-combine
a205c22 
Added filtering on search query for search bar component
@philliparaujo
Added UserGuide template and filled in my information
a351cde
@KesterTan
unverify api tests
e1705fb
@philliparaujo
Merge pull request #30 from CMU-313/user-guide
bf98345 
Added UserGuide template and filled in some information
@KesterTan
Merge pull request #32 from CMU-313/verified-tests
72ab338 
UnVerify API Tests
@KesterTan
added user guide, permissions feature
75f0ec8
@KesterTan
removed dump file
bd9ad77
@KesterTan
added newline for posttools
27250fe
@KesterTan
Merge pull request #34 from CMU-313/unverify-message
df3ac08 
Set up permissions for verify and unverify and UI for unverify
@katcday
Merge branch 'CMU-313:f24' into answer-post-ui
6aedc3c
@katcday
Added Answer Question Button
a9ac00c 
I edited the theme to add a dropdown button that can toggle the topic as answered or unanswered. It currently uses the lock/unlock backend.
@katcday
Added API to mark a topic as answered
9b7f092
@katcday
Adding Frontend Answer Functionality
0e274c1
@katcday
Fixing Lint Issue
c044d16
@katcday
Fixing lint error
49897d4
@katcday
Fixed unanswered test case
2216c0a
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11286207961

Details

  • 101 of 126 (80.16%) changed or added relevant lines in 21 files are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.03%) to 82.638%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/api/posts.js 15 16 93.75%
src/posts/answer.js 10 11 90.91%
src/topics/create.js 2 3 66.67%
src/privileges/posts.js 3 5 60.0%
src/socket.io/index.js 4 6 66.67%
src/topics/tools.js 14 16 87.5%
src/socket.io/topics/tags.js 1 8 12.5%
src/controllers/write/posts.js 7 16 43.75%
Files with Coverage Reduction New Missed Lines %
src/api/search.js 5 72.34%
Totals Coverage Status
Change from base Build 11205295170: -0.03%
Covered Lines: 22410
Relevant Lines: 25693
💛 - Coveralls

@philliparaujo philliparaujo deleted the answer-post-ui branch November 15, 2024 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@katcday @coveralls @philliparaujo @heyanuja @KesterTan @yerimsong