one-network-integration-ui

Clone Tools

Stats

Commits this week: 3
Total Committers: 4
Last Commit:
Commits this week: 3

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Today
9 hrs ago

Krzysztof Pasiek

Released one-network-integration-ui v0.0.1-RC2

Options
    • -5
    • +5
    /docker-compose.yml
    • -1
    • +1
    /project.properties
9 hrs ago

Krzysztof Pasiek

Revert "Released one-network-integration-ui v0.0.1-RC1"

Options
    • -5
    • +5
    /docker-compose.yml
    • -1
    • +1
    /project.properties
Thursday 14 Oct
1:07 pm

Krzysztof Pasiek

Released one-network-integration-ui v0.0.1-RC1

Options
    • -5
    • +5
    /docker-compose.yml
    • -1
    • +1
    /project.properties
Monday 27 Sep
9:05 pm

Paweł Muchowski

8:02 am

Krzysztof Pasiek

Saturday 25 Sep
10:00 pm

Paweł Muchowski

OLMIS-7397: Revert changes made in tests
OLMIS-7397: Revert changes made in tests
9:11 pm

Paweł Muchowski

OLMIS-7397: Revert changes made in tests

This reverts commit af04eab1f2f24e0c91c452483b1dc20f9eca0f43.

Options
    • -3
    • +0
    /CHANGELOG.md
    • -0
    • +15579
    /npm-shrinkwrap.json
    • -4
    • +4
    /src/admin-one-network-integration/admin-one-network-integration.controller.spec.js
    • -1
    • +1
    /src/admin-one-network-integration/admin-one-network-integration.routes.spec.js
Wednesday 15 Sep
6:49 am

Paweł Muchowski

Wednesday 08 Sep
6:08 am

Krzysztof Pasiek

Tuesday 07 Sep
3:02 pm

Paweł Muchowski

OLMIS-7314: Fix tests after karma upgrade
OLMIS-7314: Fix tests after karma upgrade
2:57 pm

Paweł Muchowski

1:34 pm

Paweł Muchowski

OLMIS-7314: Fix tests after karma upgrade

Options
    • -0
    • +3
    /CHANGELOG.md
    • -4
    • +4
    /src/admin-one-network-integration/admin-one-network-integration.controller.spec.js
    • -1
    • +1
    /src/admin-one-network-integration/admin-one-network-integration.routes.spec.js
Monday 06 Sep
11:43 am

Krzysztof Pasiek

Thursday 26 Aug
9:20 am

Paweł Muchowski

OLMIS-7314: Add Webpack and migrate Grunt build to Webpack
OLMIS-7314: Add Webpack and migrate Grunt build to Webpack
Wednesday 25 Aug
2:21 pm

Paweł Muchowski

OLMIS-7314: Update scss files to enable webpack build

Options
    • -0
    • +3
    /CHANGELOG.md
    • -1
    • +0
    /npm-shrinkwrap.json
    • -1
    • +1
    /package.json
    • -1115
    • +270
    /yarn.lock
Thursday 29 Jul
7:22 am

Paweł Muchowski

Wednesday 28 Jul
3:00 pm

Paweł Muchowski

I think you can rename this to processBufferedData

I think you can rename this to processBufferedData

2:59 pm

Paweł Muchowski

you can remove "this"

you can remove "this"

2:55 pm

Paweł Muchowski

you can remove "this"

you can remove "this"

2:40 pm

Paweł Muchowski

as above

as above

2:40 pm

Paweł Muchowski

you don't need to use this

you don't need to use this

2:38 pm

Paweł Muchowski

synchronized will not be needed here if you use synchronized list

synchronized will not be needed here if you use synchronized list

2:37 pm

Paweł Muchowski

I think you should use Collections.synchronizedList()

I think you should use Collections.synchronizedList()

2:29 pm

saleksandra <soltys.ola@gmail.com>

OLMIS-7312: Refactored code after changes on backend

Options
    • -23
    • +23
    /src/admin-one-network-integration/admin-one-network-integration.controller.js
    • -20
    • +20
    /src/admin-one-network-integration/admin-one-network-integration.controller.spec.js
    • -1
    • +1
    /src/admin-one-network-integration/admin-one-network-integration.html
    • -2
    • +2
    /src/admin-one-network-integration/admin-one-network-integration.routes.js
    • -10
    • +10
    /src/admin-one-network-integration/admin-one-network-integration.routes.spec.js
    • -4
    • +4
    /src/admin-one-network-integration/messages_en.json
    • -32
    • +0
    /src/one-network-integration-scheduler/one-network-integration-scheduler.module.js
    • -97
    • +0
    /src/one-network-integration-scheduler/one-network-integration-scheduler.service.js
    • -106
    • +0
    /src/one-network-integration-scheduler/one-network-integration-scheduler.service.spec.js
    • -0
    • +32
    /src/one-network-integration/one-network-integration.module.js
    • -0
    • +93
    /src/one-network-integration/one-network-integration.service.js
    • -0
    • +106
    /src/one-network-integration/one-network-integration.service.spec.js
9:43 am

Paweł Muchowski

I don't think we need to return the status here, because it will always be true it can be void and you can use @ResponseStatus(HttpStatus.OK) to return the status

I don't think we need to return the status here, because it will always be true
it can be void and you can use @ResponseStatus(HttpStatus.OK) to return the status

9:42 am

Paweł Muchowski

you don't need to return anything here, the response it's not used in the extension you can use the @ResponseStatus(HttpStatus.OK) annotation to return the status

you don't need to return anything here, the response it's not used in the extension
you can use the @ResponseStatus(HttpStatus.OK) annotation to return the status

9:34 am

Paweł Muchowski

The status will always be false, so I don't this it's needed

The status will always be false, so I don't this it's needed

9:02 am

Paweł Muchowski

I'm not sure, it would be better if we had some other controller (config or integration controller) and we could move it there and enabling/disabling scheduler would be done in a service and called...

I'm not sure, it would be better if we had some other controller (config or integration controller) and we could move it there and enabling/disabling scheduler would be done in a service and called in the new controller

8:59 am

Paweł Muchowski

ok, but it's not fetching the products, it's receiving them

ok, but it's not fetching the products, it's receiving them

8:34 am

Paweł Muchowski

I think we should change the name of this method, it should indicate that it will clear the existing queue

I think we should change the name of this method, it should indicate that it will clear the existing queue