[13.0][MIG] github_connector: Migration to 13.0

[Tardo]

cc @tecnativa TT20653

[Tardo]

Ready to review

[ernestotejeda]

@Tardo Do you also get those errors when you test it locally or do I have to do some configuration or something extra?

[pedrobaeza]

Please remove #44, that it's insecure.

[Tardo]

Commits ordered and suggested changes applied!

I can't test the functionality properly because my account use 2FA. @ernestotejeda please, check if you get the same errors. In my test i only get "max tries reached".

[ernestotejeda]

I took this PR, so main changes are:

• Adapt python model code
• Adapt view
• Remove #44
• Convert smart-button type-action to smart-button type-object (https://github.com/OCA/interface-github/pull/47#pullrequestreview-350402397)
• Remove some wizards because I didn't see that they were used anywhere, perhaps it is old code that has been carried until now.
• Reduce the commit history a bit

[ernestotejeda]

Ready to be reviewed

[ernestotejeda]

I removed group_by key from context of the actions that are returned in a python method

[ernestotejeda]

ping @pedrobaeza

[ernestotejeda]

Changes done

[ernestotejeda]

changes done @chienandalu

[ernestotejeda]

@chienandalu , @Tardo when you have a chance, update your review status :blush:

[ernestotejeda]

Changes done

[Tardo]

Ok! changes done... tested on local and works fine. I think that this module need a "revamp" in Odoo 14.0 to use modern Github API's (GraphQL API v4).

[pedrobaeza]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it! Prepared branch 13.0-ocabot-merge-pr-47-by-pedrobaeza-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 81f93d967558d9270102b5f5c21f2b1e84b271bd. Thanks a lot for contributing to OCA. ❤️