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

[website] Trigger new Link updates #1840

Closed
wants to merge 3 commits into from

Conversation

OchiengPaul442
Copy link
Contributor

Summary of Changes (What does this PR do?)

  • Updated the link on the explore model to point to the latest AirQo analytics

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 Feb 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1446135) 7.79% compared to head (07ef330) 37.90%.
Report is 2 commits behind head on staging.

❗ Current head 07ef330 differs from pull request most recent head 8070041. Consider uploading reports for the commit 8070041 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           staging    #1840       +/-   ##
============================================
+ Coverage     7.79%   37.90%   +30.10%     
============================================
  Files          168       59      -109     
  Lines        12035      591    -11444     
  Branches         0      116      +116     
============================================
- Hits           938      224      -714     
+ Misses       11097      350    -10747     
- Partials         0       17       +17     

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

Copy link
Contributor

github-actions bot commented Feb 5, 2024

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.

Hi @OchiengPaul442 , thanks for this.

I have just noticed that the "Air Quality Analytics" still points to the old Analytics. What does this mean? Perhaps the .envs have not yet been updated?

@OchiengPaul442
Copy link
Contributor Author

Hi @Baalmart,
Thanks for your quick review,
@BenjaminSsempala updated the secretes and requested I raise a PR to trigger the change. But please feel free to confirm the changes made to the secrets.
I will send the updated URLs to your Slack DM now.

Copy link
Contributor

github-actions bot commented Feb 8, 2024

New Website changes available for preview here

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