Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commite82a874

Browse files
committed
chore: don't require an organization to read starter templates
1 parentfab1960 commite82a874

File tree

10 files changed

+54
-25
lines changed

10 files changed

+54
-25
lines changed

‎coderd/coderd.go‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -871,7 +871,7 @@ func New(options *Options) *API {
871871
r.Route("/templates",func(r chi.Router) {
872872
r.Post("/",api.postTemplateByOrganization)
873873
r.Get("/",api.templatesByOrganization())
874-
r.Get("/examples",api.templateExamples)
874+
r.Get("/examples",api.templateExamplesByOrganization)
875875
r.Route("/{templatename}",func(r chi.Router) {
876876
r.Get("/",api.templateByOrganizationAndName)
877877
r.Route("/versions/{templateversionname}",func(r chi.Router) {
@@ -915,6 +915,7 @@ func New(options *Options) *API {
915915
apiKeyMiddleware,
916916
)
917917
r.Get("/",api.fetchTemplates(nil))
918+
r.Get("/examples",api.templateExamples)
918919
r.Route("/{template}",func(r chi.Router) {
919920
r.Use(
920921
httpmw.ExtractTemplateParam(options.Database),

‎coderd/templates.go‎

Lines changed: 29 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -821,7 +821,8 @@ func (api *API) templateDAUs(rw http.ResponseWriter, r *http.Request) {
821821
// @Param organization path string true "Organization ID" format(uuid)
822822
// @Success 200 {array} codersdk.TemplateExample
823823
// @Router /organizations/{organization}/templates/examples [get]
824-
func (api*API)templateExamples(rw http.ResponseWriter,r*http.Request) {
824+
// @Deprecated Use /templates/examples instead
825+
func (api*API)templateExamplesByOrganization(rw http.ResponseWriter,r*http.Request) {
825826
var (
826827
ctx=r.Context()
827828
organization=httpmw.OrganizationParam(r)
@@ -844,6 +845,33 @@ func (api *API) templateExamples(rw http.ResponseWriter, r *http.Request) {
844845
httpapi.Write(ctx,rw,http.StatusOK,ex)
845846
}
846847

848+
// @Summary Get template examples
849+
// @ID get-template-examples
850+
// @Security CoderSessionToken
851+
// @Produce json
852+
// @Tags Templates
853+
// @Success 200 {array} codersdk.TemplateExample
854+
// @Router /templates/examples [get]
855+
func (api*API)templateExamples(rw http.ResponseWriter,r*http.Request) {
856+
ctx:=r.Context()
857+
858+
if!api.Authorize(r,policy.ActionRead,rbac.ResourceTemplate.AnyOrganization()) {
859+
httpapi.ResourceNotFound(rw)
860+
return
861+
}
862+
863+
ex,err:=examples.List()
864+
iferr!=nil {
865+
httpapi.Write(ctx,rw,http.StatusInternalServerError, codersdk.Response{
866+
Message:"Internal error fetching examples.",
867+
Detail:err.Error(),
868+
})
869+
return
870+
}
871+
872+
httpapi.Write(ctx,rw,http.StatusOK,ex)
873+
}
874+
847875
func (api*API)convertTemplates(templates []database.Template) []codersdk.Template {
848876
apiTemplates:=make([]codersdk.Template,0,len(templates))
849877

‎coderd/templateversions_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1097,17 +1097,17 @@ func TestPreviousTemplateVersion(t *testing.T) {
10971097
})
10981098
}
10991099

1100-
funcTestTemplateExamples(t*testing.T) {
1100+
funcTestStarterTemplates(t*testing.T) {
11011101
t.Parallel()
11021102
t.Run("OK",func(t*testing.T) {
11031103
t.Parallel()
11041104
client:=coderdtest.New(t,nil)
1105-
user:=coderdtest.CreateFirstUser(t,client)
1105+
_=coderdtest.CreateFirstUser(t,client)
11061106

11071107
ctx,cancel:=context.WithTimeout(context.Background(),testutil.WaitLong)
11081108
defercancel()
11091109

1110-
ex,err:=client.TemplateExamples(ctx,user.OrganizationID)
1110+
ex,err:=client.StarterTemplates(ctx)
11111111
require.NoError(t,err)
11121112
ls,err:=examples.List()
11131113
require.NoError(t,err)

‎codersdk/templates.go‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -472,9 +472,16 @@ type AgentStatsReportResponse struct {
472472
TxBytesint64`json:"tx_bytes"`
473473
}
474474

475-
// TemplateExamples lists example templates embedded in coder.
476-
func (c*Client)TemplateExamples(ctx context.Context,organizationID uuid.UUID) ([]TemplateExample,error) {
477-
res,err:=c.Request(ctx,http.MethodGet,fmt.Sprintf("/api/v2/organizations/%s/templates/examples",organizationID),nil)
475+
// TemplateExamples lists example templates available in Coder.
476+
//
477+
// Deprecated: Use StarterTemplates instead.
478+
func (c*Client)TemplateExamples(ctx context.Context,_ uuid.UUID) ([]TemplateExample,error) {
479+
returnc.StarterTemplates(ctx)
480+
}
481+
482+
// StarterTemplates lists example templates available in Coder.
483+
func (c*Client)StarterTemplates(ctx context.Context) ([]TemplateExample,error) {
484+
res,err:=c.Request(ctx,http.MethodGet,"/api/v2/templates/examples",nil)
478485
iferr!=nil {
479486
returnnil,err
480487
}

‎site/src/api/api.ts‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,7 @@ class ApiMethods {
483483
};
484484

485485
deleteToken=async(keyId:string):Promise<void>=>{
486-
awaitthis.axios.delete("/api/v2/users/me/keys/"+keyId);
486+
awaitthis.axios.delete(`/api/v2/users/me/keys/${keyId}`);
487487
};
488488

489489
createToken=async(
@@ -1754,12 +1754,8 @@ class ApiMethods {
17541754
/**
17551755
*@param organization Can be the organization's ID or name
17561756
*/
1757-
getTemplateExamples=async(
1758-
organization:string,
1759-
):Promise<TypesGen.TemplateExample[]>=>{
1760-
constresponse=awaitthis.axios.get(
1761-
`/api/v2/organizations/${organization}/templates/examples`,
1762-
);
1757+
getTemplateExamples=async():Promise<TypesGen.TemplateExample[]>=>{
1758+
constresponse=awaitthis.axios.get(`/api/v2/templates/examples`);
17631759

17641760
returnresponse.data;
17651761
};

‎site/src/api/queries/templates.ts‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -112,13 +112,10 @@ export const setGroupRole = (
112112
};
113113
};
114114

115-
exportconsttemplateExamples=(organizationId:string)=>{
115+
exportconsttemplateExamples=()=>{
116116
return{
117-
queryKey:[
118-
...getTemplatesByOrganizationQueryKey(organizationId),
119-
"examples",
120-
],
121-
queryFn:()=>API.getTemplateExamples(organizationId),
117+
queryKey:["templates","examples"],
118+
queryFn:()=>API.getTemplateExamples(),
122119
};
123120
};
124121

‎site/src/pages/CreateTemplatePage/ImportStarterTemplateView.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export const ImportStarterTemplateView: FC<CreateTemplatePageViewProps> = ({
3131
const{multiple_organizations:organizationsEnabled}=
3232
useFeatureVisibility();
3333
const[searchParams]=useSearchParams();
34-
consttemplateExamplesQuery=useQuery(templateExamples("default"));
34+
consttemplateExamplesQuery=useQuery(templateExamples());
3535
consttemplateExample=templateExamplesQuery.data?.find(
3636
(e)=>e.id===searchParams.get("exampleId")!,
3737
);

‎site/src/pages/CreateTemplatesGalleryPage/CreateTemplatesGalleryPage.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { StarterTemplatesPageView } from "./StarterTemplatesPageView";
1111

1212
constCreateTemplatesGalleryPage:FC=()=>{
1313
const{ experiments}=useDashboard();
14-
consttemplateExamplesQuery=useQuery(templateExamples("default"));
14+
consttemplateExamplesQuery=useQuery(templateExamples());
1515
conststarterTemplatesByTag=templateExamplesQuery.data
1616
?// Currently, the scratch template should not be displayed on the starter templates page.
1717
getTemplatesByTag(removeScratchExample(templateExamplesQuery.data))

‎site/src/pages/StarterTemplatePage/StarterTemplatePage.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { StarterTemplatePageView } from "./StarterTemplatePageView";
88

99
constStarterTemplatePage:FC=()=>{
1010
const{ exampleId}=useParams()as{exampleId:string};
11-
consttemplateExamplesQuery=useQuery(templateExamples("default"));
11+
consttemplateExamplesQuery=useQuery(templateExamples());
1212
conststarterTemplate=templateExamplesQuery.data?.find(
1313
(example)=>example.id===exampleId,
1414
);

‎site/src/pages/TemplatesPage/TemplatesPage.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export const TemplatesPage: FC = () => {
1111

1212
consttemplatesQuery=useQuery(templates());
1313
constexamplesQuery=useQuery({
14-
...templateExamples("default"),
14+
...templateExamples(),
1515
enabled:permissions.createTemplates,
1616
});
1717
consterror=templatesQuery.error||examplesQuery.error;

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp