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

Created maintenance page. #1775

Merged
merged 3 commits into from
Jan 10, 2024
Merged

Created maintenance page. #1775

merged 3 commits into from
Jan 10, 2024

Conversation

OchiengPaul442
Copy link
Contributor

@OchiengPaul442 OchiengPaul442 commented Dec 16, 2023

Summary of Changes (What does this PR do?)

  • Developed a none interactive Maintenance Page for enhanced user experience, providing timely updates as we diligently work on resolving backend issues.

Status of maturity (all need to be checked before merging):

  • I've tested this locally
  • I consider this code done
  • This change ready to hit production in its current state

Screenshots (optional)

image

Copy link

codecov bot commented Dec 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ff9d778) 8.05% compared to head (5999cb4) 36.96%.

❗ Current head 5999cb4 differs from pull request most recent head 988eecb. Consider uploading reports for the commit 988eecb to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           staging    #1775       +/-   ##
============================================
+ Coverage     8.05%   36.96%   +28.90%     
============================================
  Files          164       59      -105     
  Lines        11508      606    -10902     
  Branches         0      115      +115     
============================================
- Hits           927      224      -703     
+ Misses       10581      365    -10216     
- Partials         0       17       +17     

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

Copy link
Contributor

New Website changes available for preview here

Copy link
Collaborator

@Baalmart Baalmart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @OchiengPaul442 , how does one navigate to this page? Can we be able to make the entire website's landing page point to this maintenance page by just making a small change in the website code? Or just deploying it separately as an App Engine instance and just redirecting traffic to this page whenever one accesses airqo.net? cc: @Psalmz777

@Baalmart
Copy link
Collaborator

Baalmart commented Jan 9, 2024

Hi @OchiengPaul442 , I hope this finds you well.
Just sharing a brief note about this PR (subject) below....

So we recently had a meeting with @Psalmz777 where we agreed to have a new folder in our frontend project where we shall create a SINGLE file frontend project without any framework --- just simple plain HTML one page which we shall deploy to App Engine SEPARATELY.

WHY THIS DIRECTION?

  • We did realise that some of the challenges with using frameworks is the reason we are even creating a maintenance page.
  • We do not want to reach a level where we also need to create a maintenance page for a maintenance page! :)
  • Lastly, we need to be able to ensure that our maintenance will always be readily available regardless of any circumstances. One of the ways for this is to ensure that it does not have dependencies on any framework (Django, etc) or DB technology. Just simple plain HTML + Javascript code in a "single file" project should be enough.

@OchiengPaul442 OchiengPaul442 changed the title [WEBSITE] Created a maintenance page Created a maintenance page. Jan 9, 2024
@OchiengPaul442 OchiengPaul442 changed the title Created a maintenance page. Created maintenance page. Jan 9, 2024
@Baalmart Baalmart merged commit c00dbdc into staging Jan 10, 2024
27 checks passed
@Baalmart Baalmart deleted the website-downtime branch January 10, 2024 08:29
@Baalmart Baalmart mentioned this pull request Jan 10, 2024
3 tasks
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.

2 participants