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

Removed module Mage_Poll #3098

Merged
merged 1 commit into from
Mar 19, 2023
Merged

Removed module Mage_Poll #3098

merged 1 commit into from
Mar 19, 2023

Conversation

fballiano
Copy link
Contributor

@fballiano fballiano commented Mar 19, 2023

From my experience the Mage_Poll module was almost never used by anybody, and in 2023 I think it really is unnecessary in our core, this PR removes it.

If you agree with this PR I'll create a separate repository for the module (as we did for Mage_Backup), so that users that still want can install it.

NOTE: PHPStan errors are not related to this PR, they appear only because this PR targets branch 20.0 and PHPStan's baseline is broken on branch 20.0.

@github-actions github-actions bot added Component: Adminhtml Relates to Mage_Adminhtml Component: Poll Relates to Mage_Poll Template : admin Relates to admin template Template : base Relates to base template translations Relates to app/locale XML Layout labels Mar 19, 2023
@addison74
Copy link
Contributor

I often encountered the situation when stores wanted to find out the opinion of visitors. Here comes this module to offer a solution. For example, in a store, the delivery company chosen to ship the orders created big problems and the complaints multiplied. The shop owner made this aspect public through a message accompanied by a poll and asked for the vote. Following the result, he had discussions with the forwarder and gave up his services. Today, very few stores keep in touch with customers in a real way about problems. Most have the impression that if they have better prices, discount coupons, the business is milk and honey.

I have nothing against removing the source code from the distribution as long as the respective modules are kept in a repository.

@fballiano
Copy link
Contributor Author

let's say nowadays there are other tools like chat or surveys connected to the newsletters

@justinbeaty
Copy link
Contributor

I definitely don't care what my customer's favorite color is, let's remove it!

In general I'd support the removal of any module that isn't essentially core functionality, moving that code to a separate repo, and optionally including it in the release builder.

@fballiano
Copy link
Contributor Author

@justinbeaty I've targeted only v20, since I hope we'll approve the new release schedule/branches I don't think it's worth doing it in v19

@justinbeaty
Copy link
Contributor

@fballiano agreed. I think the last v19.5.0 release is good for v19 for getting Zend etc out of the repo, but I wouldn't want to spend much time on v19 either.

Copy link
Member

@elidrissidev elidrissidev left a comment

Choose a reason for hiding this comment

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

IMG_20230319_144504.jpg

@fballiano
Copy link
Contributor Author

@fballiano fballiano merged commit b695f93 into OpenMage:20.0 Mar 19, 2023
@fballiano fballiano deleted the remove_poll branch March 19, 2023 16:55
@addison74
Copy link
Contributor

I have nothing against removing this module. I find that the opinions are the majority in this regard. Please just put the module files in a repository for those who want to use it in the future. Basically it is a module that works and has not generated any issues according to reports so far.

@fballiano
Copy link
Contributor Author

creating the repo for Mage_Poll now, actually I'll have to update the README here too.

@fballiano
Copy link
Contributor Author

@kiatng
Copy link
Contributor

kiatng commented Mar 20, 2023

I am not against this PR. However, I think for a PR that remove more than a few files should wait for at least 3 days before merging. So that community members living in other time zones or not-working can have a chance to voice an opinion.

@sreichel
Copy link
Contributor

+1 for removal. -1 for v20-only :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Adminhtml Relates to Mage_Adminhtml Component: Poll Relates to Mage_Poll Template : admin Relates to admin template Template : base Relates to base template translations Relates to app/locale XML Layout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants