Skip to content

Conversation

@freshteapot
Copy link
Contributor

Summary

Making copy to clipboard work in Safari

@freshteapot
Copy link
Contributor Author

@pavsaund pavsaund self-requested a review April 13, 2022 11:55
@pavsaund pavsaund force-pushed the copy-to-clipboard-why-so-hard branch from 8187808 to c4848bc Compare May 11, 2022 06:37
const msName = ms.name;

const copyPOHeadUrl = async (event: React.MouseEvent<HTMLElement>) => {
const copyPOHeadUrl = (event: React.MouseEvent<HTMLElement>) => {

Choose a reason for hiding this comment

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

Similar blocks of code found in 4 locations. Consider refactoring.

};

const copyPOLineUrl = async (event: React.MouseEvent<HTMLElement>) => {
const copyPOLineUrl = (event: React.MouseEvent<HTMLElement>) => {

Choose a reason for hiding this comment

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

Similar blocks of code found in 4 locations. Consider refactoring.

const webhookPoLine = 'm3/poline';

const copyPOHeadUrl = async (event: React.MouseEvent<HTMLElement>) => {
const copyPOHeadUrl = (event: React.MouseEvent<HTMLElement>) => {

Choose a reason for hiding this comment

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

Similar blocks of code found in 4 locations. Consider refactoring.

};

const copyPOLineUrl = async (event: React.MouseEvent<HTMLElement>) => {
const copyPOLineUrl = (event: React.MouseEvent<HTMLElement>) => {

Choose a reason for hiding this comment

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

Similar blocks of code found in 4 locations. Consider refactoring.

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit c4848bc and detected 4 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 4

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants