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.

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

Add UI for displaying question status as 'Resolved' or 'Not Resolved' #32

Closed
wants to merge 10,000 commits into from

Conversation

luciafang
Copy link

Description:
I implemented a UI feature in the General Discussion section that displays the resolved status of questions, based on backend logic. If a question is answered, it is marked as 'Resolved'. If not, it is marked as 'Not Resolved'.
The resolved status is displayed next to topics in the topic list.

I made the following changes:

  1. {{{if}}}, {{{else}}}, {{{end}}}. for display backend logic
  2. Added logic to display a status badge next to each topic in the topic list.
  3. If the question is resolved, display a green "Resolved" badge. Else, display a gray "Not Resolved" badge.

Overview:
The resolved status badge is visible in the topic list, making it easier for users to identify which discussions have been resolved at a glance.

Related Issues:
#1 #2

How to Test:
Navigate to the General Discussion section.
For topics that have been answered, you should see a green "Resolved" badge.
For topics without answers, a gray "Not Resolved" badge should be displayed.

Resolves #5

barisusakli and others added 30 commits April 15, 2024 12:10
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…2518)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
if merge was started from inside the topic
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
luciafang and others added 28 commits September 4, 2024 19:59
…ognitive complexity from 18 to 15; created a check duration test file named as checkDuration.js; passed all test cases
project 1 refactoring deep function calls
add 'resolved' boolean field to topic data
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10983425813

Details

  • 15 of 39 (38.46%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.8%) to 81.826%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/flags.js 3 4 75.0%
public/src/client/topic/postTools.js 5 28 17.86%
Totals Coverage Status
Change from base Build 10688272652: -0.8%
Covered Lines: 22343
Relevant Lines: 25863

💛 - Coveralls

@luciafang luciafang closed this Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants