digital-land / submit

0 stars 1 forks source link

Notify LPAs that their brownfield land data needs updating #616

Closed CharliePatterson closed 1 week ago

CharliePatterson commented 3 weeks ago

Context:

ToDo

additional

Tests to write

Rules for when the banner should show:

Acceptance criteria:

Designs:

Image

GeorgeGoodall-GovUk commented 3 weeks ago

@CharliePatterson @MJFaucherFolie For now I'll just consider their latest BFL endpoint, though perhaps we can talk about this more in the meeting later

GeorgeGoodall-GovUk commented 2 weeks ago

@alextea @maddie-broxup Hey both, this ticket also involves adding in an overdue notice.

for now I've just put in this while I'm finishing of development. Image

could you guys let me know how exactly you want it? thanks

maddie-broxup commented 2 weeks ago

Hey @GeorgeGoodall-GovUk, have attached an image of the content below.

384329208-f2b6b3ab-1b3e-47f9-916b-cfeb8e905d06

I added it to ticket #638 and not this one, sorry!

Heading: Your brownfield land dataset is overdue Paragraph text: It was due on 31 December 2024. Link text: Follow the steps and check your data meets the specifications before you publish and submit it.

GeorgeGoodall-GovUk commented 2 weeks ago

awesome, thanks @maddie-broxup I'll put that in now

GeorgeGoodall-GovUk commented 2 weeks ago

also one more thing, how will this reflect in the hint under the card?

Image

maddie-broxup commented 2 weeks ago

@GeorgeGoodall-GovUk The text under the 'Brownfield land' link should say: You must update this dataset by 31 December 2024

GeorgeGoodall-GovUk commented 2 weeks ago

@maddie-broxup even when overdue? or should the message be different for that?

maddie-broxup commented 2 weeks ago

Ah sorry @GeorgeGoodall-GovUk yeah you're right - can we make this "This dataset is overdue"?

GeorgeGoodall-GovUk commented 2 weeks ago

sounds good! thanks :D

GeorgeGoodall-GovUk commented 2 weeks ago

extending this ticket slightly as we want this on the BFL dataset details page too now

Image

neilfwar commented 1 week ago

Spilling over due to merge conflicts causing test fail - likely completed 20 Nov