Merge main to react testing so we may test it with database functions #20
Conversation
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
Made some changes to assigned tasks good stuff!
Done! Next, we gotta add the creation script.
Locust databasemodelling
V1 mostly done, might need more changes, such as standardizing the names, but, still, mostly done.
FK constraints now are named TABLENAME_FOREIGNTABLE_FK_NUMIFNEEDED UK and PK constraints follow the same name standard. Might be easier to alter them if needed... Also, attribute names are now named after their use inside the table and/or named the same as their origin table instead of having FK_TABLENAME_attributename standard. Easier to reference when inserting values and such.
finishing up this mess. Next should be the pop script, the most boring ass thing to write.
The database creation, deletion and population scripts are ready for testing in the app. Also wrote some design document on how the hell should we program a messaging system.
can't have something bad written and not point out it's terrible, eh?
Scripts moved inside source, basic user setup written. db.js now is not so empty anymore, still very basic. Will commence tests today, I do believe. Not so sure, though.
link pro miro
…stem This entire branch should be mostly done! All that's left is more testing and it should be ready to merge into main.
simple as
Added database basics
Added info on meeting held today.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Precisely what the title says.
Gotta merge this with react branch to keep up the changes in the database scripts.