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

B6 - elara health final #428

Merged
merged 4 commits into from
Jan 27, 2025

Conversation

jameshnl232
Copy link
Contributor

I optimized the images and make some small changes. It should work now!!!!!!

@jameshnl232 jameshnl232 reopened this Jan 24, 2025
@bkleinen
Copy link
Contributor

Sie haben meine 2 Änderungen rückgängig gemacht:

1.:
-demo_link = "https://elara-health.vercel.app/"
+website_link = "https://elara-health.vercel.app/"
-> wenn ich das richtig sehe, ist der Link keine Website, die Ihr Projekt beschreibt, sondern ihre App selbst. Dafür gibt es den demo_link.

2: ich hatte in diesen Satz "often" eingefügt. da ich vermute, dass Sie es rauslöschen weil Sie nicht auf der letzten Version in main aufbauen, und es eher ein versehen ist, meine Änderungen zu überschreiben, erwähne ich es nochmal....

  • ME/CFS (Myalgische Enzephalomyelitis/Chronic Fatigue Syndrome) affects millions of people worldwide, yet it remains poorly understood and often stigmatized.
  • ME/CFS (Myalgische Enzephalomyelitis/Chronic Fatigue Syndrome) affects millions of people worldwide, yet it remains poorly understood and stigmatized.

War das Absicht? Wenn ja, warum?

@bkleinen
Copy link
Contributor

ich habs mir noch genauer angesehen, sie bauen auf der letzten Version auf, (branch base), aber Sie überschreiben meine Änderungen aktiv im commit f52bd64

  • Absicht oder haben Sie Ihre Datei drüberkopiert?
    (Note: git diff/patches)

@jameshnl232
Copy link
Contributor Author

jameshnl232 commented Jan 24, 2025

Ich werde es korrigieren

@jameshnl232 jameshnl232 reopened this Jan 24, 2025
@jameshnl232
Copy link
Contributor Author

Die Link ist tatsächlich die Webseite, wir haben nur noch keine offiziel domain.

@bkleinen bkleinen merged commit a78d1e1 into htw-imi-showtime:main Jan 27, 2025
22 checks passed
@bkleinen
Copy link
Contributor

Die Link ist tatsächlich die Webseite, wir haben nur noch keine offiziel domain.

ich habe jetzt einen neuen link type eingeführt: app_link - das sollte dann besser passen :)
Sie können auch gerne im Text nochmal explizit zum Ausprobieren der App einladen. vlt. ist es dafür ja auch ganz gut, eine Test/Demo instanz zu haben.

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.

2 participants