Skip to content

[ADD] Ask for confirmation when copying records - #847

Merged
moylop260 merged 1 commit into
OCA:8.0from
EssentNovaTeam:feature/8.0/web_copy_confirm
Feb 6, 2018
Merged

moylop260 merged 1 commit into
OCA:8.0from
EssentNovaTeam:feature/8.0/web_copy_confirm

Conversation

@StefanRijnhart

Copy link
Copy Markdown
Member

No description provided.

@StefanRijnhart StefanRijnhart added this to the 8.0 milestone Jan 26, 2018
@pedrobaeza

Copy link
Copy Markdown
Member

Microsoft syndrome? 😆

No, it's common to press there instead of "Delete", so good idea!

"category": "Tools",
"depends": [
'web',
],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks, added now!

@legalsylvain legalsylvain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks.

@StefanRijnhart
StefanRijnhart force-pushed the feature/8.0/web_copy_confirm branch from 1647064 to a16ae9a Compare January 26, 2018 12:29

@moylop260 moylop260 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍
Tested on runbot

@moylop260
moylop260 merged commit f077376 into OCA:8.0 Feb 6, 2018
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.

5 participants