Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Secure API Handling: Remove Hardcoded Key and Update Docs for Key Retrieval #1

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Secure API Handling: Remove Hardcoded Key and Update Docs for Key Retrieval #1

wants to merge 12 commits into from

Conversation

altanapps
Copy link

This PR removes an exposed API key from the main repository. To ensure the continued functionality and security of our codebase, the following changes have been made:

  • The hardcoded API key has been purged to prevent unauthorized access.
  • Documentation has been updated with clear instructions for users on how to obtain and manage their own API keys.
  • A minor cleanup was also performed to remove unnecessary binary files, maintaining the cleanliness of the repository.

altanapps and others added 12 commits December 17, 2023 13:14
@altanapps
fix: removes the API key in the Github repository and replaces that w…
d14d1d7 
…ith appropriate README.md for reproducibility
@altanapps
deleted the unnecessary binary files to ensure main repo cleanliness
3b2a61b
@altanapps
feature: pull the binance data and write it into the local file
bca1c93
@altanapps
fix: delete the .env file
c0780ae
@altanapps
fix: added .DS_Store to the .git_ignore file
b13d050
@altanapps
process the csv file to clearly define the labelling for what to predict
4546a51
@altanapps
enhancement: add two more labels for the data
14da5f9
@altanapps
fix: read dataset from the NEAR data, ideally add new features in reg…
ab875b8 
…ards to the BTC price
@altanapps
separated the data into train/val features and labels
50678c5
@altanapps
enhancement: able to process data for the NEAR dataset
fbf8956
@altanapps
Merge pull request #1 from altanapps/feature/dataset
4a7c5a9 
Feature/dataset
@altanapps
Merge pull request #2 from altanapps/feature/processing-dataset
bb6d27a 
Feature/processing dataset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@altanapps