Skip to content

Rework find-peers #29

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
Open
wants to merge 77 commits into
base: main
Choose a base branch
from
Open

Rework find-peers #29

wants to merge 77 commits into from

Conversation

n-mattos
Copy link
Contributor

  • Switched database from JSON to SQLite
  • Implemented filters in intra synchronization
  • Replaced 42connector with Fast42

n-mattos and others added 30 commits August 19, 2025 13:21
@n-mattos
Updated gitignore
c1a93bc
@n-mattos
Change json database into sqlite
e0f70b3
@n-mattos
Added prisma migration
d0ec237
@n-mattos
Does not store entire user profile anymore, now only stores an access…
855cc2e 
… token for authentication
@n-mattos
Removed unused database functions
f0c52f9
@n-mattos
Added transform, insert, and query functions using SQLite and Prisma
d8c3cdc
@n-mattos
Removed unused types
7f508ac
@n-mattos
Added functions for fetching data from the 42Api using fast42
9682763
@n-mattos
Installed fast42
69b72a1
@n-mattos
Added variables to limit fetching during development
bb2a313
@n-mattos
Added functions to synchronize the database with Intra
b1f8913
@n-mattos
IDs will now be stored in the SQLite database
6625713
@n-mattos
Now saves the SyncTimestamp in the SQLite database, instead of a file…
128873b 
…. The functions are now also part of the DatabaseService class
@n-mattos
Adjusted msUntilNextPull according to the new database
80bcc29
@n-mattos
Added comment
e15ba5a
@n-mattos
Added new database using SQLite, replacing JSON.
fe86578
@n-mattos
Added front-end to match the new database. Also fixed the database in…
83c7a8b 
…sert/query functions
@n-mattos
Increased readability
87f34ad
@n-mattos
Gave the site an updated (Codam Inspired) look
9497b8d
@n-mattos
Added a short site description at the top of the page
bc9be27
@n-mattos
Removed unnecessary variable
29b1d8d
@n-mattos
Added an option to hide empty projects
b67e471
@n-mattos
Removed dev filter (forced to only fetch campus 14)
dcfa45f
@n-mattos
Retries syncing multiple times before throwing an error. + Filter gho…
73f9191 
…st campus out.
@n-mattos
Project Users now fully sync one page of project users and its depend…
41fceab 
…encies before fetching the next page
@n-mattos
Function namechange, not a callback function
69dabfb
@n-mattos
Filters project users to only process updated ones.
1ebc956
@n-mattos
Trailing whitespace
9c95fdf
@n-mattos
Store which campuses contain which projects. This allows the site to …
5d0afcf 
…only display projects that are present on the given campus
@n-mattos
Fixed syntax error on getting primary campus
61545f2
SirMorfield
SirMorfield reviewed Sep 18, 2025
View reviewed changes
src/wrapper.ts Outdated
let i = 0;
const pageItems = await Promise.all(pages.map(async (page) => {
let p = null;
pageFetch: while (!p) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh no, I thought that we did not allow gotos
I think you should try to write this function without this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

@n-mattos
campus to project relation removed, project difficulty added.
4ab1bda
@n-mattos
Adjusted prisma schema according to previous commit
aedacd1
@n-mattos
Past migration doesn't need to be pushed (it's a very outdated databa…
bdc7f56 
…se by now)
@n-mattos
Removed image caching proxy
c4dc419
@n-mattos
Filter Ghost Campus out of dropdown in the error page
c3954d6
@n-mattos
Removed label in while loop
f535d00
@n-mattos
Removed source from the footer
c23ff59
@n-mattos
Filter projectuser fetching on 42 cursus and piscine
5955514
@n-mattos
Add slug back for project links. Moved Ghost campus to id 42
b3ba78c
@n-mattos
Fix regarding slug
44a60b6
@n-mattos
Increase fetch timeout to 60s
bd9a4f4
@n-mattos
Added slug into DisplayProject interface
0838127
@n-mattos
Changed sync. New order: projects->projectuser->user->campus. And onl…
374ca26 
…y cursus 21 and 9 are synced, instead of every cursus
@n-mattos
Removed 'go-to'
93f3319
@n-mattos
Added anonymization, by clearing the database of anonymized users aft…
f3ada19 
…er every synchronization
@n-mattos
Stores user info at authentication, to prevent constant api calls fro…
c9ac5c3 
…m active users
@n-mattos
Fixed new authentication
ef13ce4
@n-mattos
Replaced all space indentations with tabs, and added some ;
ddf17b6
@n-mattos
Added semicolons
727332f
@n-mattos
More semicolons
f486fa0
@n-mattos
Removed unused functions, and added comments to new ones
d6b5056
@n-mattos
Added staff check for login
c7b9b4e
@FreekBes FreekBes linked an issue Sep 30, 2025 that may be closed by this pull request
Find Peers not working with 503 Service Temporarily Unavailable #30
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SirMorfield SirMorfield SirMorfield requested changes

@FreekBes FreekBes FreekBes requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Find Peers not working with 503 Service Temporarily Unavailable
3 participants
@n-mattos @SirMorfield @FreekBes