Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow GitHub URLs to be set in settings to support GitHub Enterprise and Enterprise Cloud #27

Open
jonruiz opened this issue May 26, 2023 · 5 comments · May be fixed by #53
Open

Allow GitHub URLs to be set in settings to support GitHub Enterprise and Enterprise Cloud #27

jonruiz opened this issue May 26, 2023 · 5 comments · May be fixed by #53
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@jonruiz
Copy link

jonruiz commented May 26, 2023

Tartelet hardcodes the Server and API URLs to GitHub.com. It would be nice if these could be overridden in settings to support GitHub Enterprise and Enterprise cloud.

@jonruiz
Copy link
Author

jonruiz commented May 26, 2023

I tried to do this locally. I was able to get around signing issues by changing the Bundle Identifier, then ran into issues with keychain access, but will keep it on the radar.

artelet is awesome. Nice work!

@simonbs
Copy link
Contributor

simonbs commented Jul 17, 2023

@jonruiz This is a great idea! I'm happy to review a PR that adds this.

@simonbs simonbs added enhancement New feature or request good first issue Good for newcomers labels Jul 17, 2023
@mattia
Copy link
Contributor

mattia commented Aug 31, 2023

I tried to do this locally. I was able to get around signing issues by changing the Bundle Identifier, then ran into issues with keychain access, but will keep it on the radar.

artelet is awesome. Nice work!

Regarding the keychain issue, did you also update the keychain implementation in CompositionRoot.swift (here)? At the moment it has hardcoded the value, after updating it I was able to have the project running without issues

@jonruiz
Copy link
Author

jonruiz commented Sep 2, 2023

Ah yes thank you. That works for me too. Much appreciated!

@mattia mattia linked a pull request Oct 4, 2023 that will close this issue
@mattia
Copy link
Contributor

mattia commented Oct 4, 2023

Hi! 👋
If you are still interested I've opened a (very much) draft PR (#53) that should work. You are welcome to test it, but you will need to compile your own Tartelet version.
Let me know if it works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants