digitales
/
kompass
4
1
Fork 1
Code Issues 45 Pull Requests 2 Packages Projects Releases Wiki Activity

Labels Milestones

feat(startpage): add link model for external links on admin startpage #138

Merged
christian.merten merged 4 commits from MK/startpage_gallery into main 10 months ago
Conversation 7 Commits 4 Files Changed 8
marius.klein commented 10 months ago
Collaborator
There is no content yet.
christian.merten reviewed 10 months ago
christian.merten left a comment
Owner

Sollen die ganzen logos hier noch sein? Ich dachte, dass die jetzt im Link model verschwinden?

Sollen die ganzen logos hier noch sein? Ich dachte, dass die jetzt im `Link` model verschwinden?
jdav_web/templates/admin/index.html Outdated
@ -103,3 +202,3 @@
<tr>
<td>
<a href="{% settings_value 'CLOUD_LINK' %}">Nextcloud</a>
Jugendreferent*in
christian.merten commented 10 months ago
Owner

Ich bin etwas verwirrt: Ist die Idee nicht, dass das jetzt allles dynamisch aus dem Link model gezogen wird? Hier hardcodest du aber jede Menge Sachen.

Das wäre btw. auch ein möglicher temporärer fix, wir können einfach in der lokalen Heidelberg config dieses template überschreiben.

Ich bin etwas verwirrt: Ist die Idee nicht, dass das jetzt allles dynamisch aus dem `Link` model gezogen wird? Hier hardcodest du aber jede Menge Sachen. Das wäre btw. auch ein möglicher temporärer fix, wir können einfach in der lokalen Heidelberg config dieses template überschreiben.
christian.merten commented 10 months ago
Owner

Ich bin etwas verwirrt: Ist die Idee nicht, dass das jetzt allles dynamisch aus dem Link model gezogen wird? Hier hardcodest du aber jede Menge Sachen.

Das wäre btw. auch ein möglicher temporärer fix, wir können einfach in der lokalen Heidelberg config dieses template überschreiben.

Ah, ich glaube ich habe es teilweise verstanden. Link ist nur für links (...), nicht für Ansprechpersonen.

Wir sollten dennoch nicht alle Kontakt email adressen hier hardcoden. Du könntest hier einfach

{% include startpage/contact.html %}

machen und in startpage/contact.html den alten Inhalt reinkopieren. Dann können wir in der lokalen Heidelberg config einfach das startpage/contact.html template überschreiben mit deiner neuen Version.

> Ich bin etwas verwirrt: Ist die Idee nicht, dass das jetzt allles dynamisch aus dem `Link` model gezogen wird? Hier hardcodest du aber jede Menge Sachen. > > Das wäre btw. auch ein möglicher temporärer fix, wir können einfach in der lokalen Heidelberg config dieses template überschreiben. Ah, ich glaube ich habe es teilweise verstanden. `Link` ist nur für links (...), nicht für Ansprechpersonen. Wir sollten dennoch nicht alle Kontakt email adressen hier hardcoden. Du könntest hier einfach ``` {% include startpage/contact.html %} ``` machen und in `startpage/contact.html` den alten Inhalt reinkopieren. Dann können wir in der lokalen Heidelberg config einfach das `startpage/contact.html` template überschreiben mit deiner neuen Version.
christian.merten added the
Status
Awaiting author
 label 10 months ago
marius.klein commented 10 months ago
Poster
Collaborator

ich hab das umgesetzt und in prod- und dev-config die Sektions-spezifischen Kontaktdaten hinzugefügt.

ich hab das umgesetzt und in prod- und dev-config die Sektions-spezifischen Kontaktdaten hinzugefügt.
christian.merten commented 10 months ago
Owner

Was ist mit den ganzen Logos? Ich dachte die werden dynamisch im Link model hinzugefügt?

Was ist mit den ganzen Logos? Ich dachte die werden dynamisch im `Link` model hinzugefügt?
marius.klein commented 10 months ago
Poster
Collaborator

genau, aktuell ist es so implementiert, dass diese Icons als image für den entsprechenden Link hochgeladen werden können. Daduch bestehen gar keine Probleme mit Bildrechten, da keine Bilder im Repo eingecheckt werden.

genau, aktuell ist es so implementiert, dass diese Icons als image für den entsprechenden Link hochgeladen werden können. Daduch bestehen gar keine Probleme mit Bildrechten, da keine Bilder im Repo eingecheckt werden.
christian.merten commented 10 months ago
Owner

Aber dein PR fügt jede Menge Bilder hinzu!

Aber dein PR fügt jede Menge Bilder hinzu!
marius.klein commented 10 months ago
Poster
Collaborator

sorry, wer Augen im Kopf hat, sollte die auch benutzen 🙈 habs korrigiert und die Bilder ganz aus der History entfernt.

sorry, wer Augen im Kopf hat, sollte die auch benutzen 🙈 habs korrigiert und die Bilder ganz aus der History entfernt.
christian.merten reviewed 10 months ago
christian.merten left a comment
Owner

Danke, wenn du die zwei Sachen noch änderst kann ich es noch bis nachher deployen.

Danke, wenn du die zwei Sachen noch änderst kann ich es noch bis nachher deployen.
jdav_web/startpage/models.py Outdated
@ -104,0 +121,4 @@
'image/gif'])
visible = models.BooleanField(verbose_name=_('Sichtbar'), default=True)
christian.merten commented 10 months ago
Owner

Kannst du hier noch ein

class Meta:
    verbose_name = _('Link')
    verbose_name_plural = _('Links')

hinzufügen? Dann steht im Admin "Link hinzufügen" statt "link hinzufügen". Von mir aus auch ohne Übersetzung, hier ist eh Deutsch = Englisch. Musst dann nur noch eine weitere Datenbankmigration machen (und squashen)

Kannst du hier noch ein ``` class Meta: verbose_name = _('Link') verbose_name_plural = _('Links') ``` hinzufügen? Dann steht im Admin "Link hinzufügen" statt "link hinzufügen". Von mir aus auch ohne Übersetzung, hier ist eh Deutsch = Englisch. Musst dann nur noch eine weitere Datenbankmigration machen (und squashen)
christian.merten marked this conversation as resolved
jdav_web/startpage/templates/startpage/contact.html Outdated
@ -0,0 +15,4 @@
<td>
<a href="mailto:{% settings_value 'RESPONSIBLE_MAIL' %}">jugendreferat@jdav-hd.de</a>
</td>
</tr>
christian.merten commented 10 months ago
Owner

Hier fehlt ein

</table>
</div>
Hier fehlt ein ``` </table> </div> ```
christian.merten marked this conversation as resolved
christian.merten commented 10 months ago
Owner

sorry, wer Augen im Kopf hat, sollte die auch benutzen 🙈 habs korrigiert und die Bilder ganz aus der History entfernt.

Btw: die history verschwindet eh, weil ich immer squash merge, das heißt alles was in die finale history geht ist die finale version mit der PR beschreibung als commit message.

> sorry, wer Augen im Kopf hat, sollte die auch benutzen 🙈 habs korrigiert und die Bilder ganz aus der History entfernt. Btw: die history verschwindet eh, weil ich immer squash merge, das heißt alles was in die finale history geht ist die finale version mit der PR beschreibung als commit message.
marius.klein commented 10 months ago
Poster
Collaborator

achso, gute Info, danke!
Hab die zwei Dinge angepasst.

fyi: ich wusste nicht, dass man beim squashen der migrations nicht nochmal den gleichen Namen nehmen sollte, er hat sich beschwert, es aber trotzdem gemacht, ich hoffe das funktioniert jetzt trotzdem.

achso, gute Info, danke! Hab die zwei Dinge angepasst. fyi: ich wusste nicht, dass man beim squashen der migrations nicht nochmal den gleichen Namen nehmen sollte, er hat sich beschwert, es aber trotzdem gemacht, ich hoffe das funktioniert jetzt trotzdem.
christian.merten removed the
Status
Awaiting author
 label 10 months ago
christian.merten changed title from Links im Kachel-Format für die interne Startseite to feat(startpage): add link model for external links on admin startpage 10 months ago
christian.merten merged commit c4d6581e6f into main 10 months ago
christian.merten deleted branch MK/startpage_gallery 10 months ago
The pull request has been merged as c4d6581e6f.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels   
Effekt
Admins
Betrifft die Administration des Kompass   
Effekt
Anwender
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins   
Effekt
Infrastruktur
Betrifft die Entwicklungs- und Produktiv Infrastruktur
  
Issue gut geeignet für Einsteiger:innen

Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt   
Kind/Breaking

Breaking change that won't be backward compatible   
Kind/Bug

Something is not working   
Kind/Documentation

Documentation changes   
Kind/Enhancement

Improve existing functionality   
Kind/Feature

New functionality   
Kind/Security

This is security issue   
Kind/Testing

Issue or pull request related to testing
  
Lokal
Heidelberg
Betrifft die Kompass Produktivumgebung von Heidelberg   
Lokal
Ludwigsburg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
  
Priority
Critical
The priority is critical   
Priority
High
The priority is high   
Priority
Low
The priority is low   
Priority
Medium
The priority is medium
  
Reviewed
Confirmed
Issue has been confirmed   
Reviewed
Duplicate
This issue or pull request already exists   
Reviewed
Invalid
Invalid issue   
Reviewed
Won't Fix
This issue won't be fixed
  
Status
Abandoned
Somebody has started to work on this but abandoned work   
Status
Awaiting author
The pull request or issue awaits a response by the author to questions or comments by reviewers.   
Status
Blocked
Something is blocking this issue or pull request   
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Effekt
Admins
Effekt
Anwender
Effekt
Infrastruktur
Issue gut geeignet für Einsteiger:innen
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Lokal
Heidelberg
Lokal
Ludwigsburg
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Awaiting author
Status
Blocked
Status
Need More Info
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: digitales/kompass#138
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'MK/startpage_gallery'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes