Skip to content

feat: add profiles#49

Closed
CLEMENTINATOR wants to merge 1 commit intomainfrom
profiles
Closed

feat: add profiles#49
CLEMENTINATOR wants to merge 1 commit intomainfrom
profiles

Conversation

@CLEMENTINATOR
Copy link
Copy Markdown
Member

No description provided.

@CLEMENTINATOR CLEMENTINATOR requested a review from a team as a code owner April 7, 2026 17:01
Copy link
Copy Markdown
Member

@SachaMorard SachaMorard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CLEMENTINATOR I'm not convinced by this PR. We need user profile management (I often need it myself when developing), but it seems to me that adding management of Edgee team specificities in this open source repo should be avoided. Why not use local aliases to manage environments, rather than doing it globally for all users?

@CLEMENTINATOR
Copy link
Copy Markdown
Member Author

@CLEMENTINATOR I'm not convinced by this PR. We need user profile management (I often need it myself when developing), but it seems to me that adding management of Edgee team specificities in this open source repo should be avoided. Why not use local aliases to manage environments, rather than doing it globally for all users?

Local aliases won't help with the credentials being different between environments, but I understand your point !
Maybe it could also be used later when the users can switch between the oss gateway and the managed services ?
I don't know, but feel free to close the PR if you feel it's not relevant !

@SachaMorard
Copy link
Copy Markdown
Member

@CLEMENTINATOR let me suggest another approach in a new PR. As I need profiles to ease my benchmarks, I started to develop something, and I added the definition of the base URLs so it could solve what you wanted to solve.
#50

@SachaMorard SachaMorard closed this Apr 8, 2026
@CLEMENTINATOR CLEMENTINATOR deleted the profiles branch April 8, 2026 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants