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

move to production #1811

Merged
merged 12 commits into from
Jan 16, 2024
Merged

move to production #1811

merged 12 commits into from
Jan 16, 2024

Conversation

Baalmart
Copy link
Collaborator

@Baalmart Baalmart commented Jan 16, 2024

…rror

[Hotfix] Resolved collocation application error
@Baalmart Baalmart requested a review from Codebmk January 16, 2024 07:27
Copy link

codecov bot commented Jan 16, 2024

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (ff9d778) 8.05% compared to head (e85bba9) 37.58%.
Report is 152 commits behind head on master.

Files Patch % Lines
website/frontend/src/components/nav/TopBar.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1811       +/-   ##
===========================================
+ Coverage    8.05%   37.58%   +29.52%     
===========================================
  Files         164       59      -105     
  Lines       11508      596    -10912     
  Branches        0      115      +115     
===========================================
- Hits          927      224      -703     
+ Misses      10581      355    -10226     
- Partials        0       17       +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Baalmart
Copy link
Collaborator Author

@OchiengPaul442 / @FaithDaka / @PaulZana

Context: The Website.

  • I have been informed of some pending images from Comms since we still have placeholders.
  • On that note, this PR (code) will be moved to production BUT the Website will NOT be deployed to production until the image issue is full addressed. Thanks.

@Baalmart Baalmart merged commit 53c12e7 into master Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants