Skip to content

[16.0][MIG] attachment_synchronize: Migration to 16.0#2562

Closed
kevinkhao wants to merge 62 commits intoOCA:16.0from
akretion:16.0-mig-attachment_synchronize
Closed

[16.0][MIG] attachment_synchronize: Migration to 16.0#2562
kevinkhao wants to merge 62 commits intoOCA:16.0from
akretion:16.0-mig-attachment_synchronize

Conversation

@kevinkhao
Copy link
Contributor

depends on #2560

@kevinkhao kevinkhao force-pushed the 16.0-mig-attachment_synchronize branch 8 times, most recently from b63b46c to cd2ff35 Compare February 23, 2023 00:17
)

def setUp(self):
super().setUp()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure but probably better to use setUpClass(cls) here ?

Copy link
Member

@bealdav bealdav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some typo

@kevinkhao kevinkhao force-pushed the 16.0-mig-attachment_synchronize branch from d3e15c3 to 5b0b1e9 Compare March 3, 2023 07:31
@kevinkhao kevinkhao force-pushed the 16.0-mig-attachment_synchronize branch from 5b0b1e9 to b062309 Compare May 18, 2023 05:48
@kevinkhao kevinkhao force-pushed the 16.0-mig-attachment_synchronize branch from 313ace1 to d190801 Compare July 3, 2023 10:15
@florian-dacosta
Copy link
Contributor

Hi @kevinkhao I think you can close this one as the one with refactore has been merged some time ago

@kevinkhao
Copy link
Contributor Author

Hi @kevinkhao I think you can close this one as the one with refactore has been merged some time ago

Indeed, thanks

@kevinkhao kevinkhao closed this Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

Comments