#17 Refactoring and dividing everything into service specific modules/classes
Closed: Fixed 5 years ago by pac23. Opened 5 years ago by pac23.

Instead of the current structure,refactor everything into service specific modules making it cleaner and easier to work with.
Current structure is :

fedora-change-wrangler
./promote
../convert(issue to user story)
./announce
../mail_list.py(set user story status as well)
./fesco
../pagure_new(creates new issue in pagure)
./accept
../bugzilla_create(creates new bug in bz+issue in pagure + changes user story status in taiga)

As one might notice functionality is divided by the functionality category but all the updation at taiga,bugzilla and pagure is spread out all over the place.

I propose instead dividing it on the basis of service(taiga,pagure,bugzilla) going with the classes approach.
for example
fedora-change-wranger
./taiga
./taiga (contains all auth,conversion,status,custom attr,etc in one place)
./pagure(contains similar approach to taiga in one place)
./bugzilla(Same for bz)
main.py
config.ini

Also as we discussed we can have all the auth here itself in the same place.Code is cleaner,easier to maintain for everyone in the long run.


added in commit 2c7fe70

with further improvements in further commits.

Metadata Update from @pac23:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Log in to comment on this ticket.

Metadata