-
Notifications
You must be signed in to change notification settings - Fork 3
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
ID4 Release to Master #233
Conversation
This reverts commit 2d2c2e2.
…3-setup-tls Setup TLS
…ewanCMPT371/term-project-2024-team-3 into 228/Assertions-Docstrings-PredictedDataPage
…rtions-Docstrings-PredictedDataPage renamed usePredictedFile hook to usePredictFile
…pload-Page-QA added comments, assertions, and logging
…ewanCMPT371/term-project-2024-team-3 into 204/Login-Service-Class-Testing
…ewanCMPT371/term-project-2024-team-3 into QA/187-UpdateTestPlan
…n-Service-Class-Testing
…ewanCMPT371/term-project-2024-team-3 into QA/187-UpdateTestPlan
…pdateTestPlan Qa/187 update test plan
…omium and webkit and added more granular navbar tests
…ewanCMPT371/term-project-2024-team-3 into QA/239-Playwright-improvements
…laywright-improvements Qa/239 playwright improvements
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.
Everything is great! Just needs to add documentation.
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.
Looks good to me. Ready to merge
LGTM! |
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.
Still needs the documents added but otherwise looks good.
…/id4-release ID4 Documents
No description provided.