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

Keyword Search Backend - Khadija and Retaj #40

Open
wants to merge 2 commits into
base: f24
Choose a base branch
from
Open

Conversation

kwarraich
Copy link
Collaborator

Attempted to create a backend for Keyword search; where users can search through discussion posts using several keywords. In collaboration with Retaj

  • Created search.js under src/posts

  • Created lists.js under public/src/client/topic

  • In search.js included search functionality: Attempts to implement a search feature in the Posts module that filters posts based on a user-provided query.

  • In search.js added database integration: Attempts to utilize the database to retrieve post IDs efficiently using a sorted set, ensuring optimal performance even with larger datasets.

  • In topics.js addedTopic Creation and Infinite Scroll: Attempts to add topic creation button that allows users to create new topics using a prompt.
    -In topics.js added search and filtering: Attempts to integrate a topic search feature that allows users to search through topics using a keyword.
    Attempts to provide sorting options (e.g., by date or alphabetical order) and filtering capabilities to refine the displayed topics based on certain criteria.

This feature hopes to enhance user experience, making it easier to search through posts

Attempts to resolve Issue #22

…made some progess towards getting it funtional
@kwarraich kwarraich added bug Something isn't working enhancement New feature or request backEnd Issues with backend labels Oct 9, 2024
@kwarraich kwarraich requested a review from roma2023 October 9, 2024 11:53
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11254427411

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 82.677%

Totals Coverage Status
Change from base Build 11252958939: 0.02%
Covered Lines: 22338
Relevant Lines: 25598

💛 - Coveralls

@rsharaf-dev
Copy link
Collaborator

Reasons for this failure in implementation for Khadija & Retaj:

  • We are unsure whether the back and front end are comparable with each other. It may be the case that the back end works but we have improperly integrated the two together.
  • Maybe a small setting or configuration is missing, making it hard for the backend and frontend to talk to each other.
  • Additionally with the midterm we didn't spend that much time during the weekend (Oct 4 and Oct 5) and other work for other courses which encroached into this week (Oct 6).

@Abdallah-Abdaljalil Abdallah-Abdaljalil added this to the Sprint 2, Week 2 milestone Oct 10, 2024
@Abdallah-Abdaljalil Abdallah-Abdaljalil requested review from Abdallah-Abdaljalil and removed request for roma2023 October 10, 2024 16:53
Copy link
Collaborator

@Abdallah-Abdaljalil Abdallah-Abdaljalil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • I appreciate the detailed explanation
  • I appreciate the attempt
  • I understand the logic
  • Nice try, good work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backEnd Issues with backend bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants