openlmis-deployment

Clone Tools

Stats

Commits this week: 0
Total Committers: 30
Last Commit:
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 06 Feb
12:45 pm

Mateusz Kwiatkowski

Tuesday 05 Feb
2:42 pm

Łukasz Lewczyński and Nikodem Graczewski

12:27 pm

Paulina Mironiuk

Yes

Yes

10:50 am

Nikodem Graczewski

the UUID of the ...

the UUID of the ...

10:49 am

Nikodem Graczewski

Is the comment removed?

Is the comment removed?

8:38 am

Nikodem Graczewski

I agree. The duplicated files seem to have slight differences and are most likely coming from two different branches. Mateusz Kwiatkowski, could you fix up the review?

I agree. The duplicated files seem to have slight differences and are most likely coming from two different branches. Mateusz Kwiatkowski, could you fix up the review?

8:16 am

Paulina Mironiuk

This is already done, check it out here: https://review.openlmis.org/cru/#FEOLMIS-3601CFR-48720

This is already done, check it out here: https://review.openlmis.org/cru/#FEOLMIS-3601CFR-48720

8:12 am

Łukasz Lewczyński

duplication of files make that the review is hard to read

duplication of files make that the review is hard to read

8:12 am

Łukasz Lewczyński

missing changelog

missing changelog

8:11 am

Łukasz Lewczyński

do we have ticket for that?

do we have ticket for that?

Monday 04 Feb
9:39 pm

Mateusz Kwiatkowski

OLMIS-4373: removed filterBy option from requisitionsForConvert endpoint
OLMIS-4373: removed filterBy option from requisitionsForConvert endpoint
Thursday 31 Jan
2:17 pm

Klaudia Pałkowska

Wednesday 30 Jan
8:12 am

Łukasz Lewczyński

Tuesday 29 Jan
5:51 pm

Mateusz Kwiatkowski , Sebastian Brudziński and Paulina Mironiuk

12:52 pm

Łukasz Lewczyński

we only adjust the service so it should be treated as a patch change.

we only adjust the service so it should be treated as a patch change.

12:52 pm

Łukasz Lewczyński

we only adjust the service so it should be treated as a patch change.

we only adjust the service so it should be treated as a patch change.

Friday 25 Jan
10:04 am

Sebastian Brudziński

As discussed, this introduces a change of the contract for the search endpoint, so the version should be bumped to 13.0.0

As discussed, this introduces a change of the contract for the search endpoint, so the version should be bumped to 13.0.0

10:01 am

Sebastian Brudziński

Are there more than 2 facilities created and saved in the database? If not, this "findByIds" may as well always return all facilities and this test passes.

Are there more than 2 facilities created and saved in the database? If not, this "findByIds" may as well always return all facilities and this test passes.

7:26 am

Łukasz Lewczyński

I think we could add sorted here

I think we could add sorted here

7:25 am

Łukasz Lewczyński

Where we use this? I mean the method has been changed but I don't see usage of it in the review

Where we use this? I mean the method has been changed but I don't see usage of it in the review

7:24 am

Łukasz Lewczyński

missing changelogs in all services. Because we change response structure don't forget to update service version (if needed)

missing changelogs in all services. Because we change response structure don't forget to update service version (if needed)

Thursday 24 Jan
4:19 pm

Klaudia Pałkowska

OLMIS-3773: Updated facilities search by ids to return page instead of array
OLMIS-3773: Updated facilities search by ids to return page instead of array
Wednesday 16 Jan
6:30 pm

Mateusz Kwiatkowski

1:14 pm

Klaudia Pałkowska , Paulina Mironiuk and Łukasz Lewczyński

12:34 pm

Mateusz Kwiatkowski

I had to put it before export calls because we are setting DOCKER_HOST there which was breaking this part due to server beeing down

I had to put it before export calls because we are setting DOCKER_HOST there which was breaking this part due to server beeing down

12:29 pm

Mateusz Kwiatkowski

OLMIS-5728: added starting instance step to functional test deploy script
OLMIS-5728: added starting instance step to functional test deploy script
6:51 am

Mateusz Kwiatkowski

12:01 am

Josh Zamor

12:01 am

Josh Zamor

Also congrats on using Terraform for the first time.

Also congrats on using Terraform for the first time.

12:01 am

Josh Zamor

Looks good, thanks.

Looks good, thanks.