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

Broadcasts screen frontend #22

Merged
merged 24 commits into from
Feb 14, 2024
Merged

Conversation

SAISURYAtalla
Copy link
Contributor

This PR introduces a new feature to the application, allowing administrators to broadcast announcements to users. The broadcast feature provides a way for administrators to disseminate important information or updates efficiently to all app users.

It consists of a Broadcasts screen where the users can view the broadcasts/announcements made . It has details such as

  1. title
  2. body
  3. date

broadcast_1
broadcast_2

Requesting review @henilp105

@Aditya062003
Copy link
Collaborator

Kindly resolve the conflicts.

@SAISURYAtalla
Copy link
Contributor Author

conflicts resolved and is ready for review @Aditya062003 @henilp105

Copy link
Collaborator

@henilp105 henilp105 left a comment

Choose a reason for hiding this comment

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

Thanks @SAISURYAtalla, Resolve conflicts and comments.

@@ -252,6 +285,86 @@ class DummyStudents {
];
}

// class DummySkills {
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove redundant code.

);
}
}
// import 'package:flutter/material.dart';
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove redundant code.

);
}
}
// import 'package:flutter/material.dart';
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove redundant code.

@@ -155,4 +213,28 @@ class StudentProvider extends StateNotifier<StudentState> {
state = newState;
_logger.i("Removed student: ${student.name}");
}

// Student getStudentById(String studentId) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove redundant code.

);
}
}
// import 'package:flutter/material.dart';
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove redundant code.

@SAISURYAtalla
Copy link
Contributor Author

conflicts resolved and redundant code removed
requesting re-review @henilp105 @Aditya062003

@Aditya062003 Aditya062003 merged commit 195b6ed into OpenLake:main Feb 14, 2024
1 check passed
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.

3 participants