✨ [Feature] Add facades.Config.Set method about goravel

[hwbrzzl]

Thanks! please enrich more unit tests like the above.

[hwbrzzl]

Hi @sidshrivastav , hope you have a great weekend.

FYI, we are going to release 1.10.0 in recent days, if there is a chance that merging this feature into master will be great.

There is the v1.10.0 upgrade guide:

[sidshrivastav]

I'll push the PR by tomorrow.

[sidshrivastav]

Hey @hwbrzzl, it this something you want to add in framework, I can see the Config interface but do you want to add it for Config?

[hwbrzzl]

Oh sorry @sidshrivastav , just missed this message. I reviewed this requirement and think just changing Add(name string, configuration map[string]any) to Add(name string, configuration any) in contracts/config/config/go is ok, then we can achieve:

facades.Config.Add("test", map[string]any{"a", "b"})
facades.Config.Get("test.a") // b
facades.Config.Add("test.test1", "c")
facades.Config.Get("test.test1") // c

[hwbrzzl]

If you have interesting in this, please take a look, thanks! @sidshrivastav

[sidshrivastav]

Sure, you can assign it to me. I'll work on it during the weekend.