Skip to content

Meeting management features #42

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 18 commits into from
Closed

Meeting management features #42

wants to merge 18 commits into from

Conversation

MoBattah
Copy link

@MoBattah MoBattah commented Feb 26, 2025 •
edited
Loading

Added multiple functions including:

  • fetching meetings
  • creating meetings, dialog box form for those new meetings
  • deleting meetings
  • persistence of data through SQLite
  • script to prepopulate the database.

Please do not merge yet. I need to clean up this work. Thought the code may be helpful. It all works.
image
image

@MoBattah MoBattah mentioned this pull request Feb 26, 2025
Open
@sujithatzackriya sujithatzackriya linked an issue Feb 26, 2025 that may be closed by this pull request
Meeting History and Management Implementation #37
Open
@sujithatzackriya
Copy link
Collaborator

Hey @MoBattah
Appreciate your support. Please let me know if you need any support from my side.

@MoBattah
Copy link
Author

@sujithatzackriya Thank you! I'm cleaning it all up now. Should be ready to review and merge in a few days.

@MoBattah
Copy link
Author

image Continuing to clean up, test, build.

@MoBattah
Copy link
Author

MoBattah commented Mar 1, 2025

image

All the create meeting, save meeting, delete meeting functionality works. Editing in the text area works. Will do final clean up in order to merge this PR.

@MoBattah
Copy link
Author

MoBattah commented Mar 5, 2025 •
edited
Loading

I’ll verify the recent merge against my pull request. No apparent conflicts detected, but I’ll run full checks to confirm.

@sujithatzackriya
Copy link
Collaborator

sujithatzackriya commented Mar 5, 2025 via email

@MoBattah
Copy link
Author

MoBattah commented Mar 6, 2025

Sounds good. Thank you. Not much changes in the frontend. But please do check. Also, please change the PR target branch to devtest so that we can have a clean test environment before merging to main.

Understood, thank you.

@sujithatzackriya
Copy link
Collaborator

Closing this PR as this update is done by @athulchandroth in the branch feat/save-meetings-db-ui-change.

thank you for showing interest in contributing @MoBattah

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MoBattah MoBattah

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Meeting History and Management Implementation
2 participants
@MoBattah @sujithatzackriya