SAML-Toolkits / python3-saml

MIT License
688 stars 306 forks source link

Merge settings.json and advanced_settings.json #345

Closed colinet closed 1 year ago

colinet commented 1 year ago

It would make sense to merge the settings parameters of files :

into a single file such as settings.json.

Just a cosmetic proposal.

pitbulk commented 1 year ago

In your project, you are able to use them unified if you want.

It is just a cosmetic stuff, as you indicated, but unifying them right know have impact on old versions of the extension that expect both files.

colinet commented 1 year ago

Indeed, the content of advanced_settings.json may be moved into settings.json. And file advanced_settings.json may be removed. Good to know.