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

[finishes 187354275] chat on the App to ask for some information publ… #66

Merged
merged 1 commit into from
May 17, 2024

Conversation

hozayves
Copy link
Collaborator

@hozayves hozayves commented May 14, 2024

Objective

The aim is to enable public information inquiries through the chat feature on the App.

Modifications

  1. Edited server.ts.
  2. Introduced a new model named "chat".
  3. Developed an HTML file named "index.html" to evaluate the functionality of the chat.
  4. Created a chat controller for handling "getChats" and "sendFile" functions related to chat.
  5. Adjusted routes in both index.ts and the new chatRoute.ts.

Testing Guidelines

  1. Execute npm run migrate.
  2. Attempt to access the UI via http://localhost:8080/api/chats/chatApp.
  3. Additionally, login from different browsers to verify if the chat functionality allows communication.

Associated Matters

Please mention any relevant issues or pull requests.

Checklist

Ensure the following tasks are completed before submission:

  • Code adheres to the project's coding standards.
  • Changes are accompanied by relevant tests.
  • Documentation is updated, if applicable.
  • All tests pass successfully.

@hozayves hozayves added bug Something isn't working WIP labels May 14, 2024
@hozayves hozayves force-pushed the 187354275-user-should-be-able-chat-publicly branch 9 times, most recently from 57466f6 to e164015 Compare May 17, 2024 06:26
@hozayves hozayves force-pushed the 187354275-user-should-be-able-chat-publicly branch from e164015 to 0d3c320 Compare May 17, 2024 07:05
@hozayves hozayves added ready and removed bug Something isn't working WIP labels May 17, 2024
Copy link
Contributor

@JeanIrad JeanIrad left a comment

Choose a reason for hiding this comment

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

incredible work

Copy link
Contributor

@princenzmw princenzmw left a comment

Choose a reason for hiding this comment

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

Well done, bro ✌

Copy link
Collaborator

@P-Rwirangira P-Rwirangira left a comment

Choose a reason for hiding this comment

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

Great work !

@niyontwali niyontwali merged commit be31b28 into develop May 17, 2024
7 checks passed
Copy link
Collaborator

@amin-leon amin-leon left a comment

Choose a reason for hiding this comment

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

Perfect

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

Successfully merging this pull request may close these issues.

7 participants