10.0 merge pos multi session sync #467
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[ADD][DEV] pos_multi_session: allow using another sync server (e.g. local odoo)
* necessary refactoring made to be allow sync via external odoo (restored commit)
…on_sync-merge 10.0 pos multi session sync merge
[CI] new js tests
* [CI] adding product in an order test * [FIX] tests * [CI] longpoll tests, [FIX] pos_ms tests * [FIX] remained wrong definitions * [FIX] tests * [FIX] forgotten bracket * [FIX] pos_debt_notebook_sync add_channel, and in longpolling * [FIX] run_ID defenition for new orders * [FIX] broadcast message * [DEV] second server synching * [IMP] access control, removed commented code * [DOC] index.rst [REF] names of variables * [IMP] object.entries -> for (key ..) * [LINT] * [LINT] * [CI] tests for pos_ms_r * [FIX] synch widget * [FIX] tests * [FIX] tests find->_.find * [FIX] - small fixes for tests, lints, widget * [LINT], [IMP] added address argument for send functions * [LINT] * [FIX] brackets mess * [LINT] pos_ms_r * [LINT] pos_ms_r 2 * [LINT] flake 8 * [FIX] - remove delay in tests, debug=assets in urls, [FIX] endless update requests on a second server * [FIX] - adding delays in tests back * [IMP] simplifyed condition, [TEXT] enhanced comments
* improve debug logs * check for revision_error in other tests * add session name in log "runCommands: executing" * print warning text in logs on showing warning_message
…_session_sync_16 10.0 pos multi session sync 16
ilmir-k
reviewed
Nov 3, 2017
pos_multi_session/README.rst
Outdated
| POSes are able to work without synchronization, like without the module. | ||
| Instant data exchange are made via built-int longpolling feature extended by ``pos_longpolling`` module. | ||
|
|
||
| When POS becomes offline, i.e. don't have connectiont *Sync Server*, it is only able to create new orders and not allowed to modify exising orders to avoid synchronization problems. |
There was a problem hiding this comment.
When POS becomes offline, i.e. don't have connection to Sync Server, it is only able to create new orders and not allowed to modify exising orders to avoid synchronization problems.
ilmir-k
reviewed
Nov 3, 2017
pos_multi_session/doc/changelog.rst
Outdated
| @@ -3,6 +3,11 @@ | |||
| Updates | |||
| ======= | |||
|
|
|||
| `4.0.0` | |||
| ------- | |||
| - REF: Server side of synnchronization is move to a separate module ``pos_multi_session_sync`` | |||
There was a problem hiding this comment.
is moved to the separate module
dbname is used to compute channel name, but in second sync server dbname may be different from dbname of main server
|
@ilmir-k it's ready for review |
* [ADD] field fisc_pos_ids in pos.ms * [ADD] func set_fiscal_position * [ADD] trigger after changing a fiscal_position * [ADD] sync_server field on ms form * [FIX] server connection error in firefox * [LINT] * [REF] fiscal position syncing * [TEXT] * [REF] simplification
|
@yelizariev Could this fix #520 ? Seeing lots of updates here |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.