-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature 35 integrate database storage with analysis script #97
Feature 35 integrate database storage with analysis script #97
Conversation
…onfigure-scraping-parameters
…51-configure-scraping-parameters
…' into feature-35-integrate-database-storage-with-analysis-script
hello @AssahBismarkabah @stephane-segning i created a pull request on relating the analyzed data to the database |
ok then we should just wait for sirs reviews and approvals
…On Tue, Nov 28, 2023 at 10:26 AM ValantineSuh ***@***.***> wrote:
hello @AssahBismarkabah <https://github.com/AssahBismarkabah>
@stephane-segning <https://github.com/stephane-segning> i created a pull
request on relating the analyzed data to the database
—
Reply to this email directly, view it on GitHub
<#97 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BAETYIQB33BRGCH4YKGSKZDYGWU3RAVCNFSM6AAAAAA75NGKF6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRZGQYTQOBXGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot tell exactly what your changes are. So please explain it to me in the next meeting
app.py
Outdated
|
||
app = Flask(__name__) | ||
app.secret_key = 'your_secret_key' # Setting a secret key for flash messages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this secret for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this secret for?
Greetings @stephane-segning
Please Sir, concerning the above secret key, I've been trying to figure it out.
Wish to inform you that I and my team did not add any key to the app.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then remove it na
scraping.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why change this file? Is it related to your code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why change this file? Is it related to your code?
Sir we later discovered that you asked the other teams to remove the scraping parameter, which was based on the images and the text. so we had to edit the scraping functionality without specifying the specific data to extract. as seen in the function below
def scrape_website(url, depth):
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, this code should then change
app.py
Outdated
|
||
app = Flask(__name__) | ||
app.secret_key = 'your_secret_key' # Setting a secret key for flash messages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then remove it na
scraping.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, this code should then change
Ok Sir @stephane-segning |
on it sir
…On Thu, Nov 30, 2023 at 11:05 AM ValantineSuh ***@***.***> wrote:
Ok Sir @stephane-segning <https://github.com/stephane-segning>
We are working on it
—
Reply to this email directly, view it on GitHub
<#97 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BAETYIXCEN4W6KV5DMTAI6LYHBK47AVCNFSM6AAAAAA75NGKF6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTGQ2DSOBVGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
hello sir @stephane-segning we are done making the necessary changes in the app.py by importing the functions required for the database functionality |
we also removed the
which was not part of our work |
implemented functionality to connect database to analyzed data