Mark fields for non extraction

[KalobTaulien]

Closes #6031

This needs a thorough review by @TheoChevalier — I'm not 100% confident I did the right thing with the override_translatable_fields

[Pomax]

This looks fine to me but I'm going to defer to @TheoChevalier here.

dead

[TheoChevalier]

Hmm, I’m getting this error when running an inv new-env… Will have a look at the code, but it’s a bit more hard to make sure I’m not forgetting something

[KalobTaulien]

@TheoChevalier migrations are fixed, should be working fine now. Had to do some migration surgery. Should be able to run inv new-env now.

[TheoChevalier]

Thanks @KalobTaulien, working now! I’m testing PNI, and I caught a few things:

• The slug is not synchronized (tested on General product)
• We will need to synchronize a few more fields:
  • company
  • phone_number
  • live_chat
  • email
  • twitter
• The Privacy Policy link label is not translated

[KalobTaulien]

Hey @TheoChevalier thanks for catching that. I changed those to SynchronizedFields. You can see the diff in the commit here d72eade

[TheoChevalier]

@KalobTaulien Thanks, the changes are looking good, but we still need to make this label translatable and the URL field synchronized:

[KalobTaulien]

Hey @TheoChevalier Yep, I had one field but not the other in that model. I added them here: https://github.com/mozilla/foundation.mozilla.org/pull/6158/files#diff-4edb38b0ab78223309e545e5d159d907247579d18df094053b9b495fbecb82d8R167-R170

I had to go through some merge conflicts, and Im a tad worried I got the translation fields backwards for a couple. May require another full review.

I also think we should work on closing this ticket ASAP because the database migrations are leaking into this branch causing painful migration conflicts (and soaking up time that it doesn't need to). Let's work on closing this ticket and opening new smaller tickets if we need them. Thoughts on that @TheoChevalier ?

[TheoChevalier]

Hey @TheoChevalier Yep, I had one field but not the other in that model. I added them here: https://github.com/mozilla/foundation.mozilla.org/pull/6158/files#diff-4edb38b0ab78223309e545e5d159d907247579d18df094053b9b495fbecb82d8R167-R170

Thanks, but that doesn’t seem to work unfortunately… I can’t manage to have these fields appear in the French page

I had to go through some merge conflicts, and Im a tad worried I got the translation fields backwards for a couple. May require another full review.

I also think we should work on closing this ticket ASAP because the database migrations are leaking into this branch causing painful migration conflicts (and soaking up time that it doesn't need to). Let's work on closing this ticket and opening new smaller tickets if we need them. Thoughts on that @TheoChevalier ?

Yeah, totally fine with me, we can work on the privacy policy labels in a follow up ticket. Also if there’s any adjustment needed to these fields, I’ll definitely catch those during the content migration, so let’s merge this I guess?

[TheoChevalier]

R+, with the privacy policy labels issue being moved to a separate ticket

[KalobTaulien]

Perfect, thanks @TheoChevalier! I opened #6257