Group management view - Githubissues

OpenEnergyPlatform / oeplatform

Repository for the code of the Open Energy Platform (OEP) website. The OEP provides an interface to the Open Energy Family

http://openenergyplatform.org/

GNU Affero General Public License v3.0

61 stars 19 forks source link

Group management view #1611

Closed jh-RLI closed 2 months ago

jh-RLI commented 3 months ago

Summary of the discussion

Describe the findings of the discussion in the issue or meeting.

Type of change (CHANGELOG.md)

Added

Add a new functionality (#)

Updated

Update existing functionality (#)

Removed

Remove a broken link (#)

Workflow checklist

Automation

Closes #1559 Closes #1607

PR-Assignee

[x] 🐙 Follow the workflow in CONTRIBUTING.md
[x] 📝 Update the CHANGELOG.md
[ ] 📙 Update the documentation on mkdocs

Reviewer

[ ] 🐙 Follow the Reviewer Guidelines
[ ] 🐙 Provided feedback and show sufficient appreciation for the work done

jh-RLI commented 3 months ago

@bmlancien I have noticed that the icons are missing in this branch. I assume it's easy to add them back. But I will add the group management page first. Just so we don't forget to add the icons again.

bmlancien commented 3 months ago

I have noticed that the icons are missing in this branch

@jh-RLI Do you mean the icons on the tabs? If so, I hid them on smaller screens in order to take less space and not destroy the tab layout after adding a fourth tab (group management). Or do you mean something else?

jh-RLI commented 3 months ago

Ah i see. I've seen that they exist, but I didn't notice the difference on smaller screens. We could also only show the icons but it is good for now, thanks :)

bmlancien commented 3 months ago

Alright :) In general I would choose the text over the icons if it's possible, as an icon alone is not always enough to understand what it's about.

jh-RLI commented 2 months ago

This related PR https://github.com/OpenEnergyPlatform/oeplatform-postgres/pull/1 for docker users should be merged by the time this PR i smerged. I introduces a query that requires a new postgres database extension.