-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fix Object Serialization, Proposal to improve Database-Scheme, #62
Merged
Merged
Changes from 47 commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
a46f6fd
added basic classes without functionality
SirJonithan 2e1d6ad
added missing changes
SirJonithan 60d5f7d
Merge pull request #1 from riehlegroup/main
JudithWell 9870833
minor changes
SirJonithan 491831f
Implementation of relevant classes has begun, writer classes are pret…
JudithWell cf7e295
adapted ProjectManager for new database scheme and naming conventions
SirJonithan dbd0d76
updated projectFeatures for new databaseScheme
SirJonithan 02713eb
updated UserConfig for new database scheme
SirJonithan d969b70
updated server to call the correct methods
SirJonithan 7381664
Added databaseInitializer.ts (with updated scheme) and reworked proje…
JudithWell 2f5a7db
implemented fixes for CourseProject
SirJonithan 96aa543
added individual add/remove methods for projects
SirJonithan b9e1ca6
renamed projectGroup to course in all client api requests
SirJonithan 00599f7
Reworked Server db interface
JudithWell 20f398a
renamed some more client variables.
JudithWell ebec9cc
Merge branch 'fix-db' of github.com:SirJonithan/mini-meco into riehle…
SirJonithan 0d8ff9c
resolved merge-conflicts
SirJonithan b48809f
updated missing file
SirJonithan e6aa2ac
Tiny changes prior to pr of updates from origin.
JudithWell af32490
Merge pull request #2 from riehlegroup/main
JudithWell 1080825
Moved all serializer related classes into its own folder, to separate…
JudithWell dc770af
Caught remaining merge error in course.ts
JudithWell 38270b0
Made a method to get table name from class and adjusted some naming, …
JudithWell cd24a48
Merge branch 'main' into fix-db
JudithWell 88c6f06
Merge pull request #3 from SirJonithan/fix-db
JudithWell fdaefa3
Added Serialization to Admin, Course, CourseProject, ProjectMember
JudithWell 9165df3
Added ID based writeObject to Writer classes.
JudithWell 825f0d0
Extended DatabaseSerializableFactory for more Serializable Types.
JudithWell bb319cf
Implemented Reader, DatabaseReader. Especially fixed readObject id-wi…
JudithWell 7db401f
Applied Serializer in auth.ts and allowed null values to be set, read…
JudithWell 717c506
Adjusted databaseInitializer according to Serializer Pattern. Also ad…
JudithWell 6ff93c4
Added Object queueing to read related Entities recursively.
JudithWell 511ab99
Removed setId() from User, IDs should not be manually changeable!
JudithWell 29cdb50
Added wasHandled to track handled objects. Also Added further classes…
JudithWell 1c3044e
Added Project Membership types for readonly access to user_projects d…
JudithWell 5a36be5
Added Object tracking to Reader to avoid reading the same object mult…
JudithWell 2119b04
Added Project Manager to manage Project memberships. Also removed any…
JudithWell 16de85a
Moved ProjectManager out of Models
JudithWell bbda41d
Started testing. Fixed a bug caused by passing tablename as parameter.
JudithWell 88bb6e1
Fixed a bug: Typo in githubUsername attribute name.
JudithWell 7832594
Fixed two typos in ObjectHandler
JudithWell b8d8e48
Fixed a bug with asynchronicity which cause very weird behaviour.
JudithWell dc27c8a
Added a couple more managers that supply a create method for Users, C…
JudithWell bfeab09
moved projects query into CourseManager
SirJonithan 4e4a35a
removed console.log statements
SirJonithan f37eb94
fixed problem with not awaiting the result
SirJonithan a0c6470
Merge branch 'main' into main
JudithWell 56fa22f
Runs semester test aswell now
JudithWell a40abaf
Added ResultSetTypes to not have to use any anymore.
JudithWell 823776b
Removed some unused commented out code lines.
JudithWell 17ef3b0
Removed unused Codelines.
JudithWell 442207e
Merge branch 'main' into main
JudithWell 5a61144
Fixed errors created by merge.
JudithWell 6151a48
removed unnecessary request argument
SirJonithan 2ab59c1
Merge branch 'main' into main
SirJonithan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Who is that todo for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SirJonithan that was your change, please decide what we need to do here.