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

WFO Information content type #1485

Merged
merged 8 commits into from
Jul 31, 2024
Merged

Conversation

eric-gade
Copy link
Collaborator

@eric-gade eric-gade commented Jul 29, 2024

What does this PR do? 🛠️

This PR implements #1469, the creation of a new content-type called WFO Information. This content type will eventually supersede the WFO Promo type.

The changes are mostly config changes.

What does the reviewer need to know? 🤔

WFO Promo type lives! (for now)

In order to make the transition smoothly, we decided to preserve the existing WFO Promo content type for the moment, and to create a completely new content type called WFO Information. (Removal will be done in #1486)

Phone number module

We will now use the Phone number module to give us an actual typed phone number field across the system, with validation and configuration options. This field will now be used on the WFO Information content type.

Address module

We will also be using the Address module, which gives us a typed field for physical addresses with configuration options and validation on entry. This field will be used by the WFO Information content type.

Copy link
Contributor

It looks like our enabled modules has changed in this PR. Please ensure any documentation in https://github.com/weather-gov/weather.gov/tree/main/docs/dev/contributed-modules.md has been updated!

@eric-gade eric-gade marked this pull request as ready for review July 29, 2024 20:54
Copy link
Collaborator

@greg-does-weather greg-does-weather left a comment

Choose a reason for hiding this comment

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

Rock and roll

@greg-does-weather greg-does-weather merged commit 7600348 into main Jul 31, 2024
17 checks passed
@greg-does-weather greg-does-weather deleted the eg-1469-wfo-promo-update branch July 31, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants