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

Remove ENForm block #1265

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Remove ENForm block #1265

merged 1 commit into from
Dec 16, 2024

Conversation

comzeradd
Copy link
Member

@comzeradd comzeradd commented Dec 9, 2024

Following its retirement from the theme code.

This also removes the Counter block to avoid cleaning it up from ENForm related code.


Ref: https://jira.greenpeace.org/browse/PLANET-7673

Testing

@comzeradd comzeradd force-pushed the retire-enform branch 4 times, most recently from 8a325c4 to 4c816c5 Compare December 9, 2024 10:00
@comzeradd comzeradd self-assigned this Dec 9, 2024
@comzeradd comzeradd requested a review from mardelnet December 9, 2024 10:01
@comzeradd comzeradd marked this pull request as ready for review December 9, 2024 10:01
planet-4 added a commit to greenpeace/planet4-test-pandora that referenced this pull request Dec 9, 2024
/unhold edcacc91-cc6d-40ad-8fb9-f593ee10ffa1
@planet-4
Copy link
Contributor

planet-4 commented Dec 9, 2024

Test instance is ready 🚀

🌑 pandora | admin | blocks report | CircleCI | composer-local.json

⌚ 2024.12.16 13:47:55

Copy link
Contributor

@mardelnet mardelnet left a comment

Choose a reason for hiding this comment

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

I am not sure about this: should we remove the Fields_Model class, too?

Following its retirement from the theme code.

This also removes the Counter block to avoid cleaning it up from ENForm related code.

Ref: https://jira.greenpeace.org/browse/PLANET-7673
@comzeradd
Copy link
Member Author

I am not sure about this: should we remove the Fields_Model class, too?

Yes, those are all ENForm related. I had already removed the code that uses the Models classes, so I just removed those too.

planet-4 added a commit to greenpeace/planet4-test-pandora that referenced this pull request Dec 16, 2024
/unhold 07d380df-52b6-44db-bd51-ec565c5c9f49
@comzeradd comzeradd merged commit dc7dff5 into main Dec 16, 2024
12 checks passed
@comzeradd comzeradd deleted the retire-enform branch December 16, 2024 15:29
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.

3 participants