Skip to content

hide PAT from the export csv output#6

Open
jdmarques wants to merge 1 commit into
mainfrom
hide-pat-from-csv-export
Open

hide PAT from the export csv output#6
jdmarques wants to merge 1 commit into
mainfrom
hide-pat-from-csv-export

Conversation

@jdmarques
Copy link
Copy Markdown
Member

@jdmarques jdmarques commented Mar 5, 2026

Exporting as CSV currently will add the url as is, which contains the zwik PAT as plain text

@jdmarques jdmarques requested review from abloemert, Copilot and rudispr and removed request for Copilot March 5, 2026 16:50
@jdmarques jdmarques force-pushed the hide-pat-from-csv-export branch from 00aabc8 to ba516fe Compare March 5, 2026 16:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jdmarques jdmarques requested a review from Copilot March 5, 2026 17:26
@jdmarques jdmarques force-pushed the hide-pat-from-csv-export branch from ba516fe to 8151e44 Compare March 5, 2026 17:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

@rudispr rudispr left a comment

Choose a reason for hiding this comment

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

lgtm

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.

3 participants