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

[BUG] <title> Setting up a branch for working in local #254

Open
1 task done
filimannino opened this issue Jan 30, 2025 · 12 comments
Open
1 task done

[BUG] <title> Setting up a branch for working in local #254

filimannino opened this issue Jan 30, 2025 · 12 comments
Assignees
Labels
bug BUG - Something isn't working triage

Comments

@filimannino
Copy link
Collaborator

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

I tried in changing the sync-url and it seemed working. BTW I don't know what is the docker volume.
After that I got problems in running:
make dev-frontend
It gives me error which I think are related with the ending line character.
Can we please setup a quick TEAMS call today in which I share my screen so we can solve all these problems and I can start working?
Thank you.
@guilbep @charlottegiseleweil

Expected Behavior

No response

Steps To Reproduce

  1. In ubuntu environment
  2. I run this:
    mkdir -p secrets && cat < secrets/.env
    POSTGRES_USER=ccfatigue
    POSTGRES_PASSWORD=Change It!
    EOF
  3. Then I run this:
    make install
    Here I got this
    46 vulnerabilities (7 low, 19 moderate, 17 high, 3 critical)
  4. Then I run this:
    make dev-database
    make dev-backend
  5. And I got this:
    ValueError: invalid interpolation syntax in 'postgresql://ccfatigue:Change+It%21@localhost:5432/ccfatigue' at position 32

Environment

- OS: Ubuntu 24.04.1 LTS
- Node: v18.19.1
- npm: 9.2.0

Anything else?

No response

@filimannino filimannino added bug BUG - Something isn't working triage labels Jan 30, 2025
@guilbep
Copy link
Contributor

guilbep commented Feb 3, 2025

You should change 'Change It!' to a proper password @filimannino that's why it does not work.
The error is pretty straighforward though.
Is that okay ?

@guilbep guilbep closed this as completed Feb 3, 2025
@guilbep guilbep reopened this Feb 3, 2025
@filimannino
Copy link
Collaborator Author

filimannino commented Feb 3, 2025

I managed to get the site locally but running:
make dev-frontend
I do not see changes just refreshing.

During the make install process I got several errors and I downloaded all the things needed but I think I messed up something. I have changed a lot of files formatting so that I can arrive at the end of the process.
Given that, can we please have a meeting in which I share my screen and you follow me in the make install process?

@filimannino
Copy link
Collaborator Author

filimannino commented Feb 3, 2025

Nevertheless, I restarted the procedure from scratch. I have fetched a new branch. I changed the password as you said. @guilbep

mkdir -p secrets && cat <<EOF > secrets/.env
POSTGRES_USER=ccfatigue
POSTGRES_PASSWORD=manninof
EOF

Then I run:

  • make install
  • make dev-database

Everything smooth

When I run make dev-backend:

sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) connection to server at "localhost" (127.0.0.1), port 5432 failed: FATAL: password authentication failed for user "ccfatigue"

(Background on this error at: https://sqlalche.me/e/14/e3q8)

I also tried:
mannino@CCLAB-C007588:/mnt/c/Users/mannino/ccfatigue_platform/CCFatiguePlatform$ psql -U ccfatigue -h localhost
Password for user ccfatigue: manninof
psql: error: connection to server at "localhost" (127.0.0.1), port 5432 failed: FATAL: password authentication failed for user "ccfatigue"

@guilbep
Copy link
Contributor

guilbep commented Feb 3, 2025

@filimannino when changing the password, you need to recreate the database, so you should probably delete the current volume, and restart the database with the new password.

@filimannino
Copy link
Collaborator Author

filimannino commented Feb 3, 2025

Ok, i managed to and I got the site running without changing anything. Thank you @guilbep

I have run:

  • make dev-backend

I opened another shell and I put:

  • make dev-frontend

Now i have the site in local but I cannot see the changes in the .vue file clicking just the refresh button on google.

@filimannino
Copy link
Collaborator Author

filimannino commented Feb 3, 2025

In addition I tried with this simple change in the DataUpload.vue:

Mi chiamo Filippo

And i got this error:

ERROR Failed to compile with 1 error 6:01:22 PM

[eslint]
/mnt/c/Users/mannino/ccfatigue_platform/CCFatiguePlatform/frontend/src/views/DataUpload.vue
11:12 error Replace ⏎··········Mi·chiamo·Filippo⏎········ with Mi·chiamo·Filippo prettier/prettier

✖ 1 problem (1 error, 0 warnings)
1 error and 0 warnings potentially fixable with the --fix option.

You may use special comments to disable some warnings.
Use // eslint-disable-next-line to ignore the next line.
Use /* eslint-disable */ to ignore all warnings in a file.
ERROR in [eslint]
/mnt/c/Users/mannino/ccfatigue_platform/CCFatiguePlatform/frontend/src/views/DataUpload.vue
11:12 error Replace ⏎··········Mi·chiamo·Filippo⏎········ with Mi·chiamo·Filippo prettier/prettier

✖ 1 problem (1 error, 0 warnings)
1 error and 0 warnings potentially fixable with the --fix option.

webpack compiled with 1 error

Moreover I am sure that the problem is in the newline command because if I only add text the site still runs. For example:


Refer to the
TST Data Convention
to prepare your Dataset.

becomes

    <p>
      Refer to the trial
      <a :href="TSTDataConventionURL"> TST Data Convention </a>
      to prepare your Dataset.
    </p>

And everything is ok.

@guilbep
Copy link
Contributor

guilbep commented Feb 4, 2025

Hello @filimannino If you want me to have a look, you need to push your changes to your branch. That way it'll be available to everyone.

  • try to use eslint --fix like the message suggest.
  • Regarding the refreshing of the page: If you have compiling errors, that's probably the reason why it's not displayed 👍
  • "in the .vue file clicking just the refresh button on google." <-- this does not make sense to me, could you clarify ?

Also, a good rule is: One problem == One issue

@filimannino
Copy link
Collaborator Author

filimannino commented Feb 4, 2025

Ok. Regarding the refreshing of the page is not a problem of compiling. Because by only changing a word it doesn't work. @guilbep

I put this:
<p>Gaia</p>
and run make dev-frontend and everything was smooth

Then i change with this
<p>Pippo</p>
and the refreshing is not working.

@guilbep
Copy link
Contributor

guilbep commented Feb 4, 2025

@filimannino in what file ? Do you save the file ?

@filimannino
Copy link
Collaborator Author

filimannino commented Feb 4, 2025

In DataUpload.vue. Of course I save the file! @guilbep

I have pushed the branch. Is everything ok? Can you please have a look at it?

Can we please have a teams call in which you show me all the correct procedures?

Thank you.

@guilbep guilbep self-assigned this Feb 4, 2025
@guilbep
Copy link
Contributor

guilbep commented Feb 4, 2025

@filimannino I cannot see your latest commit only that you have created a branch called: Filippo_Mannino_february3rd)

I can make the changes reflect directly in the browser as you can see in the following videos, I sent you a zoom meeting so we can see what's wrong on your side

ccfatigue.webm

@filimannino
Copy link
Collaborator Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug BUG - Something isn't working triage
Projects
None yet
Development

No branches or pull requests

2 participants