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.

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

TA#67305 [16.0][MIG] report_aeroo #64

Merged
merged 20 commits into from
Oct 25, 2024
Merged
Show file tree
Hide file tree
Changes from 18 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ jobs:
name: Setup Log Folder For Reports
command: sudo mkdir -p .log && sudo chmod 777 .log

# - run:
# name: Run Test
# command: docker-compose run --rm odoo run_pytest.sh
- run:
name: Run Test
command: docker-compose run --rm odoo run_pytest.sh

# - store_test_results:
# path: .log
- store_test_results:
path: .log

# job that find the next tag for the current branch/repo and push the tag to github.
# it will trigger the publish of a new docker image.
Expand Down
1 change: 0 additions & 1 deletion .docker_files/main/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
"category": "Other",
"summary": "Install all addons required for testing.",
"depends": [
"account",
"report_aeroo",
],
"installable": True,
Expand Down
9 changes: 3 additions & 6 deletions .docker_files/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
# git+https://github.com/aeroo/aeroolib.git
# waiting for PR https://github.com/aeroo/aeroolib/pull/12
git+https://github.com/adhoc-dev/aeroolib@master-fix-ods
git+https://github.com/aeroo/currency2text.git
# use this genshi version to fix error when, for eg, you send arguments like "date=True" check this https://genshi.edgewall.org/ticket/600
git+https://github.com/numigi/aeroolib@master
Babel==2.9.1
genshi==0.7.7
freezegun==1.5.1
html2text==2024.2.26
ddt==1.2.1
ddt==1.2.1
18 changes: 13 additions & 5 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,24 @@
LABEL maintainer="contact@numigi.com"

USER root
#Install dependencies for aeroo reports
RUN apt-get update && apt-get install -y --no-install-recommends \

Check notice on line 6 in Dockerfile

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

Dockerfile#L6

Delete the apt-get lists after installing something

Check warning on line 6 in Dockerfile

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

Dockerfile#L6

Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>`
default-jre \
libreoffice-java-common \
libreoffice-writer \
poppler-utils

COPY .docker_files/requirements.txt .
RUN pip3 install -r requirements.txt
# we can't use `pip install --user` as the $HOME of odoo is a volume
# so everything that is installed in $HOME will be overwritten by the mounting.

ENV THIRD_PARTY_ADDONS /mnt/third-party-addons
RUN mkdir -p "${THIRD_PARTY_ADDONS}" && chown -R odoo "${THIRD_PARTY_ADDONS}"
COPY .docker_files/requirements.txt ./requirements.txt
RUN pip3 install -r ./requirements.txt && rm ./requirements.txt

USER odoo

COPY report_aeroo /mnt/extra-addons/report_aeroo
COPY ./report_aeroo /mnt/extra-addons/report_aeroo

COPY .docker_files/main /mnt/extra-addons/main
COPY .docker_files/odoo.conf /etc/odoo


30 changes: 0 additions & 30 deletions report_aeroo/LICENSE

This file was deleted.

88 changes: 0 additions & 88 deletions report_aeroo/README.md

This file was deleted.

Loading
Loading