Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor modules #9

Open
wants to merge 38 commits into
base: 14.0
Choose a base branch
from
Open

Refactor modules #9

wants to merge 38 commits into from

Conversation

bosd
Copy link

@bosd bosd commented Jul 29, 2022 •
edited
Loading

Creating this PR her, did a lot of refactoring.
as described in: #6

t-900-a and others added 19 commits June 6, 2022 22:14
@t-900-a @bosd
sales order unit test foundation
a5e9d5f
@t-900-a @bosd
formatting corrections
8f53a4c
@t-900-a @bosd
added sales order unit tests
7b615a1
@t-900-a @bosd
flake8 / black formatting
cabf71c
@t-900-a @bosd
flake8 / black formatting
59983c6
@t-900-a @bosd
flake8 / black formatting
5ab337b
@t-900-a @bosd
flake8 / black formatting
941f488
@t-900-a @bosd
flake8 / black formatting
2048465
@t-900-a @bosd
initial point of sale code
024a8fb
@t-900-a @bosd
security fix - payment token reuse
a2c4c5c 
payment token could be reused for the odoo website
this could allow those who know what they're doing to pay once for many orders
+ flake8 / black formatting
@bosd
Fixes internal server error on unavailable rpc
d66a914
@bosd
rename to payment_monero
f036e05
@bosd
rename to pos_payment_monero
9188a69
@bosd
add base_monero
45ceb3c
@bosd
Set development status to Alpha
fe8c5a1
@bosd
imp documentation of automatic currency rate update
19242c3
@bosd
Remove unused imports
2e54340
@bosd
Update Readme.md Fixup Travis
1e94195
@bosd
Update README.md
5083e6d 
Fix travis
@bosd bosd mentioned this pull request Jul 29, 2022
Closed
@bosd bosd mentioned this pull request Mar 12, 2024
Draft
@bosd
Copy link
Author

bosd commented Mar 13, 2024

Not all of the Todo is covered yet.
Not sure if I can do that all within a short timeframe.

Since there is new interest in this module.
Maybe better to merge this refactor as is.

So improvements can be handled in later pr's and migration of this module and it dependents can continue.

Therefore marking this pr as ready.

@bosd bosd marked this pull request as ready for review March 14, 2024 12:19
@bosd bosd changed the title [WIP] Refactor modules Refactor modules Mar 16, 2024
@bosd bosd force-pushed the 14.0 branch 2 times, most recently from 07d62c8 to 3bc37ce Compare March 17, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bosd @t-900-a