Project 3A: Static Test - Retire.js#137
Closed
newmoooon wants to merge 63 commits into
Closed
Conversation
PR: getting notifications when someone responds to your post
Refactor (src/user/data.js): reduce parameters for incrDecrUserFieldBy Requirement to merge individual edits in Project 1B to team repo
Send reply popup notification to user
Refactorgroups
Delete extra files
Add missing url to vendored nodebb-theme-harmony metadata
P1B: Starter Task: Refactoring PR Issue: P1B: Refactor (src/user/search.js:137): Function with high complexity (count = 11): filterAndSortUids
Resolved function with high complexity in bookmarks.js
User Story 3: Add Least Replied topic sorting to prioritize unanswered questions
Reduced parameter count in getTagTidsByCids This commit refactors the getTagTidsByCids function in src/topics/tags.js by reducing its parameter count from 4 to 3. Two related parameters are put into a single object to simplify the function interface and improve readability.
…ix that has a red dot emoji and [staff] to show that they are staff
Sprint2-US5-Add a staff flair. Code has been reviewed. Tests have passed.
…er with background, then change background to RED
US5 - Additional Touches to the Staff Flair. Code has been reviewed. Now merging!
…ected category only
…est and view my automated tests
US5 Spring 2 Part 1: Add automated tests for US 5 and UserGuide.md to explain testing The test is red due to the outage in Coveralls. More in my comment about proof that coverage is the only issue.
Added Category-Scoped Tag Management (Rename and Delete) in Admin Panel
…dula Display Category-Specific Tag Counts in Tag Filter
US5 Spring 2 Part 2: Staff-answered -- unset when staff delete/purge their reply + recalculate & tests
Author
|
Wrong PR repo sorry |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Concrete evidence that you have successfully installed the tool
Artifacts that demonstrate that you have successfully run the tool on your repository
Pros
Cons