IDGEN-42 Ticket progress tracker

platform-2_2
Tags: #<Tag:0x00007efd1dffb920>

(Daniel Kayiwa) #21

Such changes do not appear in our pool of pull requests which we routinely look at. Therefore, always do a pull request.


(Samuel Male) #22

@tendomart just a reminder on issuing a PR :blush:


(Samuel Male) #23

On such a freaky Friday, @tendomart would really wanna raise a PR :joy:


(Irene Nyakate) #24

How is @tendomart doing, is he able to receive these notifications? :slight_smile:


(tendo kiiza Martyn) #25

@irenyak1 , @samuel34
Wanted to folks but,my machine is faulty.Fixing it…but updates are this week

I refactored the repo because it was noisy and making some changes as some of my Mock Test Objects are still failing.so am working hard to make them pass so i can effectually raise a PR.


(Irene Nyakate) #26

Hello @tendomart how are you doing? was your machine fixed? Should we expect a PR today or tomorrow?


(tendo kiiza Martyn) #27

@irenyak1 am fixed Tomorrow hopefully


(tendo kiiza Martyn) #28

did a PR at https://github.com/openmrs/openmrs-module-idgen/pull/71 with some errors for @samuel34 's sake… .still fixing the remaining methods.


(Samuel Male) #29

Man it looks like your working from the master branch. Did you just forget?


(tendo kiiza Martyn) #30

@samuel34 oops yeah, i did.After cleaning the previous commit ,forgot to create a new branch…But look at the changes anyways will change that.


(Samuel Male) #31

I wish your work was on a different branch, I would get it locally and efficiently make more changes


(tendo kiiza Martyn) #32

let me find a way out.I hope it doesn’t get noisy again.


(tendo kiiza Martyn) #33

@c.antwi @samuel34 forked my branch ,were trying to break it down into smaller tasks to accelerate Release…