openclimatefix / quartz-frontend

Front End repo for the Nowcasting project.
https://openclimatefix.org/projects/nowcasting/
MIT License
103 stars 16 forks source link

Quartz Rebrand #376

Closed braddf closed 1 year ago

braddf commented 1 year ago

Pull Request

Description

Update logo and branding to match new Quartz name/style.

Fixes #362

How Has This Been Tested?

Tested on Vercel staging preview deployment

Checklist:

Pull Request

Description

Update logo and branding to match new Quartz name/style.

Fixes #362

How Has This Been Tested?

Merged to staging for final manual testing on Vercel staging preview deployment

UAT

Quartz-specific points

PV Forecast

National

Time

Header

GSP

Time

Title

Map

Four Hour View

National

Delta View

National

Time

Header

GSP

Time

Title

Map

Combined views

Dashboard Mode

National

GSP

Legend

Map

General

Auth

Refresh

Documentation

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nowcasting-app ✅ Ready (Inspect) Visit Preview 1 resolved Jul 12, 2023 3:33pm
nowcasting-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 12, 2023 3:33pm
peterdudfield commented 1 year ago

would you be able to do the full QA?

braddf commented 1 year ago

Will do first thing Wednesday morning @peterdudfield 👍 unless @rachel-labri-tipton fancies running through the UAT in my absence tomorrow - but I would not blame anyone for not jumping to do it... 😂

rachel-labri-tipton commented 1 year ago

hey @peterdudfield and @braddf, I'd be willing to walk through the QAT today as a pre-flight check to flag anything before @braddf does the final sign off tomorrow. Is the checklist just the list in the original issue #362?

braddf commented 1 year ago

oh good point @rachel-labri-tipton - I had forgotten there was that list on the issue also, definitely that one needs "checking off", but I was thinking of the full UAT thing we use on releases usually, a la Dashboard Mode PR for reference - is that what you meant as well @peterdudfield or..?

rachel-labri-tipton commented 1 year ago

hey @braddf, thanks, I was wondering where the full list was. Well, I'm doing a bit of probabilistic stuff this morning, but I can add the checklist(s) to the PR here and maybe check a few things off this afternoon?

peterdudfield commented 1 year ago

The full list should be in another PR like - https://github.com/openclimatefix/nowcasting/pull/364.

@braddf yea thats what I was thinking

@rachel-labri-tipton if you have some time to do this, but I dont want to distract you too much from Prob stuff

braddf commented 1 year ago

@rachel-labri-tipton @peterdudfield have just done the UAT and all looks good! Fixed a couple of bits, some of which were actually broken/missing for a long time before (maybe pre-Brad tbh haha)

Permission to merge? 🙏