electrolux-oss / infrawallet

Control your cloud costs just in the way how you control your bank accounts
https://demo.infrawallet.io
Apache License 2.0
38 stars 2 forks source link

enhancement: add default to group by #15

Closed Alok650 closed 2 weeks ago

Alok650 commented 2 months ago

Hi guys, Is it possible to add a default group by i.e. by default I want the graphs to display with group by Name (name of cloud accounts)?

gluckzhang commented 2 months ago

Hi @Alok650, you mean setting the default group by to something else rather than None right?

image

We will consider adding a plugin settings page where you can change some default values, such as this one :)

jjwedemyer commented 1 month ago

Would you also think about a config value in the app config that can serve as an instance default?

gluckzhang commented 1 month ago

Would you also think about a config value in the app config that can serve as an instance default?

Hi @jjwedemyer, sorry but I missed this comment. Do you mean the configuring default view in the app config? I am not sure if I get the meaning of instance default.

jjwedemyer commented 1 month ago

Hi @gluckzhang,

sorry I was unclear. I meant introducing either a built-time configuration or a key in the app-config.yaml setting this behavior in the way the owner wants it to. "Instance default" is referring to default settings set by the owner/operator/organization in contrast to user settings which are set by the different users and are not shared.

gluckzhang commented 1 month ago

@jjwedemyer aha, now I understand it better, thanks for the explanation! I very much agree with you about having both configurations by a user and default configurations by a site admin. I think I will first enable some configurations in app-config.yaml as they act as default ones. Later on, we will see how to provide UI for users to have their own customization :)

gluckzhang commented 2 weeks ago

Hi @Alok650 and @jjwedemyer, FYI, in https://github.com/electrolux-oss/infrawallet/releases/tag/v0.1.8 we have added some configurations for the default view.

Later on, we are going to introduce the wallet idea that enables users to create their wallets (like a view with separate filters, default group by, query period, etc.). I am closing this issue for now :)