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

Summary page Backend Functionality #94

Merged
merged 6 commits into from
May 20, 2024
Merged

Conversation

adi-lux
Copy link
Contributor

@adi-lux adi-lux commented Apr 30, 2024

Description

Created backend functionality for summary page, using the session id of current user to generate all of user's meetings.

Testing

Make an account within ZotMeet, and retrieve the User UUID so it can be referenced. Below exists dummy data that can be used and converted into the database. I personally used pgAdmin to insert data so I could personally import the data into the files. You will need to replace all references of the user_id with yours in the CSV files.

mock-data-zotmeet.zip

@adi-lux adi-lux added the enhancement New feature or request label Apr 30, 2024
@adi-lux adi-lux self-assigned this Apr 30, 2024
@adi-lux adi-lux linked an issue Apr 30, 2024 that may be closed by this pull request
@adi-lux adi-lux requested a review from KevinWu098 April 30, 2024 01:21
@MinhxNguyen7 MinhxNguyen7 requested review from MinhxNguyen7 and removed request for KevinWu098 May 6, 2024 21:54
@adi-lux adi-lux requested a review from KevinWu098 May 6, 2024 21:54
@MinhxNguyen7 MinhxNguyen7 removed the request for review from KevinWu098 May 6, 2024 21:56
Copy link
Member

@MinhxNguyen7 MinhxNguyen7 left a comment

Choose a reason for hiding this comment

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

This doesn't currently work. I merged this branch on top of mine (#102), and nothing shows up in the summary page.

I'm not sure what the problem is because I'm not seeing an error. Looking through the .csv files, I do see that you have the availability string as a binary string, which is what it should be but currently isn't, pending #100, but I can't see why that would make a difference.

adi-lux added 2 commits May 20, 2024 15:08
# Conflicts:
#	src/lib/db/databaseUtils.server.ts
#	src/lib/db/schema.ts
Copy link
Member

@MinhxNguyen7 MinhxNguyen7 left a comment

Choose a reason for hiding this comment

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

LGTM!

@MinhxNguyen7 MinhxNguyen7 merged commit 8eecf2b into main May 20, 2024
1 check passed
@KevinWu098 KevinWu098 deleted the aa/summary-page-server branch October 17, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Summary Page Backend Functionality
2 participants