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.

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

fixed errors in placing confidential info #341

Closed
wants to merge 7 commits into from
Closed

fixed errors in placing confidential info #341

wants to merge 7 commits into from

Conversation

harshakhmk
Copy link

@harshakhmk harshakhmk commented May 24, 2021

Related Issue or bug

  • place all the confidential information in .env file

Fixes: #292

Describe the changes you've made

Moved all the confidential information into environment variables and sending mail when the email and password provided by the host is Not None thereby reducing latency on the SMTPLIB server
Used python decouple module with version 3.4 to configure the environment variables

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Describe if there is any unusual behaviour of your code(Write NA if there isn't)

A clear and concise description of it.

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.

Screenshots

Original Updated
original screenshot updated screenshot

@harshakhmk
Copy link
Author

@swapnilsparsh review it, is working fine now

Copy link
Owner

@swapnilsparsh swapnilsparsh left a comment

Choose a reason for hiding this comment

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

There are some bug errors while running your code, resolve them.

@harshakhmk
Copy link
Author

harshakhmk commented May 26, 2021

There are some bug errors while running your code, resolve them.

You need to install python-decouple module
Could you exactly mention what errors are you facing in the screenshot, it is working fine for me

@harshakhmk
Copy link
Author

harshakhmk commented May 26, 2021

I have updated my branch with all latest changes in master, I guess it won't have any issues now

AttributeError: module 'main_app.views' has no attribute 'email_sent'

The issue is with the master branch, code is breaking there

@swapnilsparsh
Copy link
Owner

Resolve the conflicts

@harshakhmk
Copy link
Author

Resolve the conflicts

I have updated my commits, have a look

Copy link
Owner

@swapnilsparsh swapnilsparsh left a comment

Choose a reason for hiding this comment

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

Error while running your code.

Resolve it

@harshakhmk
Copy link
Author

I didn't get any such errors on my local branches
no-issues
everything is running fine for both my branch, master(both updated)
Also, I didn't change anything in migrations scripts and all so my branch shouldn't be getting such errors, as long as you delay, I need to keep on updating my branch to make it in sync with the master

@swapnilsparsh
Copy link
Owner

ss
I am getting this error

@harshakhmk
Copy link
Author

U were not in virtual env, activate your virtual env and pull the latest commit from my branch then try the same

@swapnilsparsh
Copy link
Owner

U were not in virtual env, activate your virtual env and pull the latest commit from my branch then try the same

I have done the same still this error is showing.

@harshakhmk
Copy link
Author

U were not in virtual env, activate your virtual env and pull the latest commit from my branch then try the same

I have done the same still this error is showing.

Have u installed all pip requirements in virtual env, I have added extra package python-decouple, run pip install -r requirements.txt again in this branch and can you tell me by invoking which view u r getting this error or you are getting this error by loading the main page itself
I am sure, the
no-issues-part2
error is not from my side, I have recently tested the code, check on your side

@swapnilsparsh
Copy link
Owner

U were not in virtual env, activate your virtual env and pull the latest commit from my branch then try the same

I have done the same still this error is showing.

Have u installed all pip requirements in virtual env, I have added extra package python-decouple, run pip install -r requirements.txt again in this branch and can you tell me by invoking which view u r getting this error or you are getting this error by loading the main page itself
I am sure, the
no-issues-part2
error is not from my side, I have recently tested the code, check on your side

I have done everything but don't know why this problem is happening.
I have checked everything on my end and it's working absolutely fine, but running your code gives me an error.

@harshakhmk
Copy link
Author

can u paste the last part of error message, it is not visible

@harshakhmk
Copy link
Author

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

@swapnilsparsh
Copy link
Owner

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

Done it already.

@harshakhmk
Copy link
Author

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

Done it already.

I don't understand why you are getting those errors, everything is perfect from my side
Putting in a lot of effort and then failing to merge a PR is a little painful and all the hard work that I put in goes in vain

@swapnilsparsh
Copy link
Owner

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

Done it already.

I don't understand why you are getting those errors, everything is perfect from my side
Putting in a lot of effort and then failing to merge a PR is a little painful and all the hard work that I put in goes in vain

I am trying my level best to resolve the error as you have invested your time and hard work in this issue and due to some bug, it is not possible to merge the PR.

@harshakhmk
Copy link
Author

harshakhmk commented Jun 2, 2021

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

Done it already.

I don't understand why you are getting those errors, everything is perfect from my side
Putting in a lot of effort and then failing to merge a PR is a little painful and all the hard work that I put in goes in vain

I am trying my level best to resolve the error as you have invested your time and hard work in this issue and due to some bug, it is not possible to merge the PR.

can you reshare the last part of the error screenshot
are u getting this error on running manage.py runserver ?

@swapnilsparsh
Copy link
Owner

You need to add all variables that I mentioned in the .env file and also your google reCAPTCHA secret key too

Done it already.

I don't understand why you are getting those errors, everything is perfect from my side
Putting in a lot of effort and then failing to merge a PR is a little painful and all the hard work that I put in goes in vain

I am trying my level best to resolve the error as you have invested your time and hard work in this issue and due to some bug, it is not possible to merge the PR.

can you reshare the last part of the error screenshot
are u getting this error on running manage.py runserver ?

The photo which I shared with you above is the only error that I am getting and it's a full photo.
I am getting this error while running python manage.py migrate.

@harshakhmk
Copy link
Author

see if this helps https://stackoverflow.com/questions/39581579/django-migrations-gives-error-when-run-separately-in-different-machines
https://stackoverflow.com/questions/61658637/getting-errors-while-trying-to-run-manage-py-migrate

Tried all these things but still issue didn't resolve.

send me the bottom part of the error that you were getting after scrolling down to the bottom
Can you delete the whole branch and the code of this feature branch and try pulling it again, something got messed up with migrations

@swapnilsparsh
Copy link
Owner

see if this helps https://stackoverflow.com/questions/39581579/django-migrations-gives-error-when-run-separately-in-different-machines
https://stackoverflow.com/questions/61658637/getting-errors-while-trying-to-run-manage-py-migrate

Tried all these things but still issue didn't resolve.

send me the bottom part of the error that you were getting after scrolling down to the bottom
Can you delete the whole branch and the code of this feature branch and try pulling it again, something got messed up with migrations

Already told you that the photo which I shared with you is the complete photo with error.
You can try recommitting the code.

@harshakhmk
Copy link
Author

see if this helps https://stackoverflow.com/questions/39581579/django-migrations-gives-error-when-run-separately-in-different-machines
https://stackoverflow.com/questions/61658637/getting-errors-while-trying-to-run-manage-py-migrate

Tried all these things but still issue didn't resolve.

send me the bottom part of the error that you were getting after scrolling down to the bottom
Can you delete the whole branch and the code of this feature branch and try pulling it again, something got messed up with migrations

Already told you that the photo which I shared with you is the complete photo with error.
You can try recommitting the code.

You were not understanding what I meant, ok leave it trying deleting the codebase in this branch and pull it again

@swapnilsparsh
Copy link
Owner

see if this helps https://stackoverflow.com/questions/39581579/django-migrations-gives-error-when-run-separately-in-different-machines
https://stackoverflow.com/questions/61658637/getting-errors-while-trying-to-run-manage-py-migrate

Tried all these things but still issue didn't resolve.

send me the bottom part of the error that you were getting after scrolling down to the bottom
Can you delete the whole branch and the code of this feature branch and try pulling it again, something got messed up with migrations

Already told you that the photo which I shared with you is the complete photo with error.
You can try recommitting the code.

You were not understanding what I meant, ok leave it trying deleting the codebase in this branch and pull it again

Still showing the same error.

@khanfarhan10
Copy link

@harshakhmk conflicts still exist.

@harshakhmk
Copy link
Author

Btw which OS were you using @swapnilsparsh

@swapnilsparsh
Copy link
Owner

Btw which OS were you using @swapnilsparsh

Windows

@github-actions
Copy link

Thank you @harshakhmk for your contribution. Looking forward for more such amazing contributions :)

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