Contributing to PrestaShop Core should become more easy

[ttoine]

Dear contributors, many Pull Requests from the community have been merged last week, and that's great. We would like that contributing becomes more and more easy over time, and I would like to have your feedback:

• What are your current difficulties to submit a Pull Request on PrestaShop Core repository?
• What should we improve?
• What documentation is missing?
• Do you have other ideas?

Please, feel free to answer and discuss 

[hakeryk2]

• Quote

  What are your current difficulties to submit a Pull Request on PrestaShop Core repository?

  Well, github is fu**ing hard for newcomers. I made a lot of improvement for PHP 7.2 but I can't submit it through GitHub because I just can't use it and I need to buy some tutorial to learn it.

[ksaan]

Hello ! Here are a very helpful doc about 1st contributions to github projects  hope it helps

https://github.com/firstcontributions/first-contributions

Here is how it starts:
"It's hard. It's always hard the first time you do something. Especially when you are collaborating, making mistakes isn't a comfortable thing. We wanted to simplify the way new open-source contributors learn & contribute for the first time.

Reading articles & watching tutorials can help, but what's better than actually doing the stuff in a practice environment? This project aims at providing guidance & simplifying the way beginners make their first contribution. If you are looking to make your first contribution, follow the steps below."

Looks promising, right  ?

[Sahnoun MN]

am newest in presatshop , i have more difficulty to adapt with , i need more documentation about the inside of class, controller relation with database same like symfony or the php itself.

it will offer for the new developer the road to use presatshop easly

Thanks

[ttoine]

are you looking for the devdocs?

https://devdocs.prestashop.com/

[Sahnoun MN]

thank you i know about this doc but is not enough, i search for more complex and details documentation

[ttoine]

that's very interesting. what kind of resources would you need, or do you miss for your project ?

you can also come and ask questions on Gitter: https://gitter.im/PrestaShop/General

[Sahnoun MN]

the offcial doc its soo general no specific for the used function or relation with database in this case you must spend ofr 6 months to be a beginner, and for the prestashop 1.7 its be so difficult to know the logic of develop and some issues dont work some of 1.6 

thnks for the gitter sugess may it will help me a little

my first problem is the use of file uploadfile in customer form (sign up) because the customerFormatter use settype 'file' as dynamic and cant get the file name the show in Admin Customer as pdf download   

Edited February 5, 2019 by Sahnoun MN (see edit history)

[ttoine]

is your problem a contribution problem ? or a problem while developing your shop?

[Sahnoun MN]

no its a developement problem, i have problem with the form of customerForm in (customerFormatter.php) while i add some new input like file type like this 

 $format['file'] = (new FormField)
            ->setName('file')
            ->setType('file')
            ->setLabel(
                $this->translator->trans(
                    'file', [], 'Shop.Forms.Labels'
                )
            )
            ->setRequired(true)
        ;

in this case how can i get name of this file(pb: cant save name in database of file variable) to used next to download this file in dashboard (backoffice - AdminCustomer) because i do it with PS1.6 but in PS1.7 cant find a way

i try to do like that for file type but isnt work -> http://prestacraft.com/adding-new-fields-to-the-registration-form-in-prestashop-1-7/

Edited February 5, 2019 by Sahnoun MN (see edit history)

[ttoine]

then, please post a question about this problem on the forum 🙂

and I guess that you are not the first with this kind of problem, maybe there are already some questions about it?

[hakeryk2]

How about making a video tutorial how to create pull request and suggest something for the newcomers? How to clone, make changes, edit them etc.

[ttoine]

There are actually plenty of videos and tutorials on the web about how to use GitHub to contribute to a project.

What would be the benefit to produce a specific video for the PrestaShop project ?

[Jack Over]

I think pull request are the way to go! If you don't know how to do this search online for it.

I think good documentation is the key to success. Just starting to look into prestashop, and the first Dec docs I find is in this form.

If creating a pull request is hard, then add a link in the documentation how to create a pull request.

Or add an option to paste your code somewhere, so a more advanced developer can create the pull request.

[zapalm]

Often, inexperienced developers do not use GitHub. And modifications from them are not needed for a serious project

[zapalm]

On 2/5/2019 at 11:05 PM, Sahnoun MN said:

the offcial doc its soo general no specific for the used function or relation with database in this case you must spend ofr 6 months to be a beginner, and for the prestashop 1.7 its be so difficult to know the logic of develop and some issues dont work some of 1.6 

The current documentation has been made for a trained reader (an experienced programmer) and this documentation is really superficial.

[ChineseNorris]

On 8/2/2019 at 7:16 AM, zapalm said:

The current documentation has been made for a trained reader (an experienced programmer) and this documentation is really superficial.

The problem with this is that you can't let the community help with pull requests since they don't know even where to start... The documentation is so superficial that you have to spend a lot of time learning how prestashop is made... I'm not saying that you must write a documentation of "how to make everything" but a well written documentation of "how things are done here" would be great. It's not even necessary a documentation, a map of how database is made and how classes, controllers, and all that stuff is connected would be a better solution.

 

[El Pipo]

Hi all,

Wow, old question (still actual?), here are just a few ideas in the February 21 context:


• Why don't I propose more pull requests ?
  • Because it takes a lot of time: your code needs to be perfect, your commit message need to convince the community... It's like a diet, the tentation to give up is huge...
  • Because some of my previous pull requests has been refused just for PSR reasons or without argumentation or discussion (that's general not only for Prestashop)
  • Because there is such a long time between the pull request (I was working on) and the beginning of a discussion/merge. If ever anyone asks for a clarification, you need to dig again into it... once again, it's time consuming.
  • Because I work for a company that don't want my job to be shared with the community (with our competitors)
• Some of us don't mind about copyright and being the author of the patch. Sometimes, it's quite easier to give a patch in a github Prestashop issue, and leave the developer team include it in the code. 
• I agree Devdoc was really superficial, but it's getting better now. On the contrary, I worry it gets a messy mess. Some information is dispatched everywhere. It would deserve a bit normalization. I agree it's a hard job. One more point on documentation: Stop thinking we are all up to date with newer technologies (the "I'm stupid" brake), do not hesitate to sum up things. There are so many new concepts in 1.7 for those coming from 1.6. Do not hesitate to say more about composer, nodes, webpack. (How Prestashop is using them? Are they used a classical way? What is specific to Prestashop?) Devdoc is getting better on this point. Hard to find the right balance between Prestashop documentation and link to those tools documentation.
• Better driven community communication/history. This pined post give a link to gitter? Does it work for you? Clicking on the link provided, I get nowhere or 404. I've discovered since that it has moved to Slack. Wow, hard to follow... Those gitter links are also in a build.prestashop.com PDF. I know it's hard because Google, Bing... record everything, but it's a pain to fall on outdated documentation/link/tools.

That's just ideas/debriefing, I do not have the feeling the Prestashop Team is not trying to do its best.


Regards,
 

[El Pipo]

Just an exemple: Imagine you want to propose a commit to add a "410 gone" status to old products. You change the files in src/Core as needed. Then before to submit the patch you need to test it. As you added a new status to your form list (see https://www.prestashop.com/forums/topic/1041966-are-translation-catalog-automatically-update/) you need to have it added to your default xlf translation files in order to get it translated and then displayed.

I didn't find anything in the documentation to produce those files. Diving into the code, I think the .tn9.yml file is involved, but I didn't find any script relying on its name.

I asked the Core developper community on the forum, the #community-help and #dev-core on slack, the PrestaShop/TranslationToolsBundle on github without any answer yet.

Maybe I'm missing something obvious, but as fo today, it's a brake (at least for me) to propose a commit.

Regards, 

[ttoine]

that's where you should spend time on GitHub.

open a pull request with your improvement, and explain the help you need to finish it if it is not yet complete. maintainers will help you there.