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

Integration/first #52

Merged
merged 25 commits into from
Jun 20, 2024
Merged

Integration/first #52

merged 25 commits into from
Jun 20, 2024

Conversation

u21598012
Copy link
Contributor

Initial front to back integration

Nevin-Thomas and others added 25 commits June 7, 2024 12:59
Also made the navbar been fighting with the configuration because when I serve the Angular app it runs properly what I have coded but when I run it with electron is shows old things
Landing page is done, Upload page is done and report page still needs some love.
…ade sure all pages are reachable

I still need to give some love to the report
Angular html component does not recognize the @ symbol had to use encoding "&#64"
…ove page load performance. It also initializes Flowbite in the AppComponent's ngOnInit() method. Fixes #123
Used a bit of flowbite and some other clean up
… for violations This commit adds lazy loading for images in the app component to improve page load performance. It also updates the routing for the violations component to include a new route for getting the email type. Additionally, it fixes a bug where the Angular HTML component did not recognize the "@" symbol and had to use encoding "&#64".
Manual upload works, adds file to uploads folder
Manual upload works, adds file to uploads folder
Uses same API endpoint
@u21598012 u21598012 requested review from Yudi-G and Yeshlen June 20, 2024 13:23
@u21598012 u21598012 self-assigned this Jun 20, 2024
@u21598012 u21598012 merged commit 06a25ef into develop Jun 20, 2024
11 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants