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

Lab 8 Pull Request #712

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Lab 8 Pull Request #712

wants to merge 1 commit into from

Conversation

YavizGuldalf
Copy link
Contributor

@YavizGuldalf YavizGuldalf commented Dec 3, 2024

Below is the instructions for this lab's pull request:

Create a PR dedicated to a checklist where you will link all work related to the checklist status: commits, wiki pages, issues, meeting notes, etc.
Specify acceptance criteria and provide examples for each checklist item to ensure clear evaluation.
Start the PR with a commit in a new branch and add commits to the branch and document all the work in the PR description and comments. Ensure that the commits include co-authors, if applicable. The description and comments should include the names of the main contributors related to the linked work.
Finalize the PR by 8:00 PM today.

Checklist items:

  • Primary features (e.g. semantic search, user management),
  • Domain-specific features (e.g. why they're specific, how they're implemented),
  • API and its documentation (e.g. endpoints, expected inputs, outputs),
  • Standard(s) being followed (e.g. its documentation, implementation),
  • Testing strategies (e.g. unit test coverage, integration testing, tools).

@yunusemreozdemir
Copy link
Contributor

yunusemreozdemir commented Dec 3, 2024

Standard(s) being followed (e.g. its documentation, implementation)

Acceptance Criteria

  • Activity Streams 2.0 should be correctly utilized and generated in the format described in the Activity Streams 2.0 Detailed Research. This should be done in related backend views.
  • Profile Follow, Post Like and Post Comment related Activity Streams should be generated in corresponding backend views.
  • A backend view to consume Activity Streams / Notifications should be implemented.
  • These Notifications should be displayed in the Frontend as a Notification Box and in the Mobile as a Notification Page.
  • Related entities in Notifications should direct users to related pages when clicked. Clicking on the actor in a notification should take the user to the actor's Profile page. Clicking on the object in Post Like / Post Comment notifications should take the user to the post's page.

Examples

Frontend Notification Box
Screenshot 2024-12-03 at 16 11 59

Mobile Notification Page
WhatsApp Image 2024-12-03 at 16 15 07

Research

Main Contributors

  • Yunus Emre Özdemir
  • Ali Tarık Şahin

W3C Standards
Issue: Research W3C Standards & Choose the most suitable one for our team/project #270
Documentation: W3C Standards

Activity Streams 2.0
Issue: Research: Activity Streams 2.0 #447
Documentation: Activity Streams 2.0 Detailed Research

Frontend

Main Contributors

  • Yunus Emre Özdemir
  • Ali Tarık Şahin
  • Elif Nur Deniz

Notifications Page
PR: Lab 5 Pull Request #478

Notifications Functionality
Issue: Make Notifications Functional #561
PR: feat(frontend): implement notifications functionality #609

Notifications Box
Issue: Create Notification Box #651
PR: feat(frontend): created notifications box, removed the original page #652

Backend

Main Contributors

  • Oktay Özel

Activity Streams Addition to Views
Issue: Activity Streams Addition to All Views #517
PR: activity_streams_model #510

Mobile

Main Contributors

  • Ahmet Oğuz Engin
  • Yağız Güldal

Notifications Page
Issue: Mobile: Implement a Notifications Page #480
PR: Lab 5 Pull Request #478

Notifications Functionality
Issue: Mobile: Notifications Backend Connection #631
PR: feat(mobile): Notifications Backend Connection #640

@alitariksahin
Copy link
Contributor

Testing Strategies
Co-Authors: @aoengin @oktayozel

@arastasci
Copy link
Contributor

API and API Documentation
co-author: @skywllker

@YavizGuldalf
Copy link
Contributor Author

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.

4 participants