openlmis-cce-ui

src

Clone Tools

Stats

Commits this week: 0
Total Committers: 12
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
Thursday 27 Jun
8:41 am

Aleksandra Sołtys

Tuesday 25 Jun
9:51 am

Łukasz Lewczyński

We decided on standup that the current version is correct

We decided on standup that the current version is correct

7:49 am

Łukasz Lewczyński

7:49 am

Łukasz Lewczyński

Nikodem Graczewski Klaudia Pałkowska Should we treat those changes are breaking changes?

Nikodem Graczewski Klaudia Pałkowska Should we treat those changes are breaking changes?

Monday 24 Jun
1:04 pm

Klaudia Pałkowska

12:53 pm

Aleksandra Sołtys

OLMIS-3176: Removed openlmis-testing-utils module.
OLMIS-3176: Removed openlmis-testing-utils module.
12:41 pm

saleksandra <soltys.ola@gmail.com>

OLMIS-3176: Removed openlmis-testing-utils module.

Options
    • -250
    • +0
    ./cce-inventory-item-details/inventory-item-details.html.spec.js
    • -30
    • +0
    ./openlmis-testing-utils/openlmis-testing-utils.module.js
    • -145
    • +0
    ./openlmis-testing-utils/template-testing-utils.factory.spec.js
  4. … 1 more file in changeset.
Wednesday 03 Apr
4:20 pm

Chongsun Ahn

8:46 am

Nikodem Graczewski

Tuesday 02 Apr
3:38 pm

Chongsun Ahn

Done.

Done.

3:37 pm

Łukasz Lewczyński

8:53 am

Chongsun Ahn

What about chained function calls, like moment.tz().format()?

What about chained function calls, like moment.tz().format()?

8:35 am

Nikodem Graczewski

Something like this: https://github.com/OpenLMIS/openlmis-ui-components/blob/master/src/openlmis-function-decorator/openlmis-function-decorator.spec.js

Something like this:
https://github.com/OpenLMIS/openlmis-ui-components/blob/master/src/openlmis-function-decorator/openlmis-function-decorator.spec.js

8:27 am

Chongsun Ahn

Can you point to some other code as an example? I had a hard enough time trying to get this test to work properly.

Can you point to some other code as an example? I had a hard enough time trying to get this test to work properly.

8:11 am

Nikodem Graczewski

Great, thanks!

Great, thanks!

8:11 am

Nikodem Graczewski

Yeah, that's perfectly correct. CurrencyService is a quite old one when we still were struggling with figuring out the promises.

Yeah, that's perfectly correct. CurrencyService is a quite old one when we still were struggling with figuring out the promises.

8:09 am

Nikodem Graczewski

Please rewrite this so it doesn't use $provide. Those utilities should be injected in the inject blocked and spied on.

Please rewrite this so it doesn't use $provide. Those utilities should be injected in the inject blocked and spied on.

Monday 01 Apr
8:28 pm

Chongsun Ahn

https://openlmis.atlassian.net/browse/OLMIS-6170

https://openlmis.atlassian.net/browse/OLMIS-6170

6:40 pm

Chongsun Ahn

If we want to do this work, let's put it in another ticket, so this ticket doesn't drag on and not finish by the end of this sprint.

If we want to do this work, let's put it in another ticket, so this ticket doesn't drag on and not finish by the end of this sprint.

Friday 29 Mar
10:17 pm

Chongsun Ahn

Also changed the currency filter in ui-components to use the new localeService.

Also changed the currency filter in ui-components to use the new localeService.

10:17 pm

Chongsun Ahn

Done.

Done.

10:16 pm

Chongsun Ahn

Reduced the duplication.

Reduced the duplication.

10:16 pm

Chongsun Ahn

Done. Hopefully this is correct. Otherwise, I'm not sure how to go about implementing this.

Done. Hopefully this is correct. Otherwise, I'm not sure how to go about implementing this.

6:56 pm

Chongsun Ahn

Then how do we catch in the locale cache code to call getLocaleSettingsFromConfig? Is there a good code example that shows this? I basically copied this from currency service.

Then how do we catch in the locale cache code to call getLocaleSettingsFromConfig? Is there a good code example that shows this? I basically copied this from currency service.

5:25 pm

Chongsun Ahn

The whole datepicker format is confusing to me as well. Does it need to be different from the date format? And where is the documentation about its meaning, vs. the date formats?

The whole datepicker format is confusing to me as well. Does it need to be different from the date format? And where is the documentation about its meaning, vs. the date formats?

4:53 pm

Chongsun Ahn

It may be a non-issue, as it looks like anything that would use datepicker would probably use business dates, which does not represent an instant in time, and so timezone would not be important. Ad...

It may be a non-issue, as it looks like anything that would use datepicker would probably use business dates, which does not represent an instant in time, and so timezone would not be important. Additionally, the datepicker directive does use the openlmis date filter.

4:35 pm

Nikodem Graczewski

I agree that it uses separate format, but what I'm concerned about is that the ticket might not pass the QA because of the second point of the acceptance criteria as the date picker will show diffe...

I agree that it uses separate format, but what I'm concerned about is that the ticket might not pass the QA because of the second point of the acceptance criteria as the date picker will show different date format (most likely one without timezone).

4:27 pm

Chongsun Ahn

Yes but the datepicker format is a separate setting, and not in the reference data. I suppose we could add it, but it wasn't done in the other tickets, so this ticket just followed what was done.

Yes but the datepicker format is a separate setting, and not in the reference data. I suppose we could add it, but it wasn't done in the other tickets, so this ticket just followed what was done.

4:12 pm

Nikodem Graczewski

I see. Isn't the purpose of this ticket to make all components use the locales from the reference data?

I see. Isn't the purpose of this ticket to make all components use the locales from the reference data?

4:12 pm

Chongsun Ahn

Alright, I'll try to do the extra checking in the localeService.

Alright, I'll try to do the extra checking in the localeService.