Skip to content

Improve documentation and data fetch script#250

Open
tnagorra wants to merge 2 commits intomainfrom
fix/data-fetch-script
Open

Improve documentation and data fetch script#250
tnagorra wants to merge 2 commits intomainfrom
fix/data-fetch-script

Conversation

@tnagorra
Copy link
Contributor

Addresses

Changes

  • Refactor fetch script
  • Make referer env variable non-mandatory

This PR doesn't introduce any

  • temporary files, auto-generated files or secret keys
  • build works
  • eslint issues
  • typescript issues
  • codegen errors
  • console.log meant for debugging
  • typos
  • unwanted comments
  • conflict markers

Comment on lines +233 to +234
console.log('CSRF Token exists:', !!csrfTokenValue);
console.log('Referer exists:', !!referer);
Copy link
Contributor

@barshathakuri barshathakuri Mar 25, 2026

Choose a reason for hiding this comment

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

Please remove these consoles.

Copy link
Member

Choose a reason for hiding this comment

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

This is intentional; having this makes it easy to identify issues with configuration.

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