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

Reloading at /member/detail/, /topics_detail/ will redirect to the top page. #86

Closed
HisashiHasebe opened this issue Jul 23, 2024 · 5 comments
Assignees

Comments

@HisashiHasebe
Copy link
Contributor

https://diverta.gyazo.com/62e71505a66cc82720355ec02ae9b48f

@VortexExpansion VortexExpansion linked a pull request Jul 23, 2024 that will close this issue
@VortexExpansion
Copy link
Contributor

@HisashiHasebe san
Can you please confirm the behaviour of the deployed site again premerge ?

@HisashiHasebe
Copy link
Contributor Author

@VortexExpansion
It cannot build.

Did you removed permissions right?
I revert /rcms-api/1/member/list permissions, because It have to set API request restriction.
Then when build, It will be error following.

"https://dev-nuxt-auth.a.kuroco.app/rcms-api/1/member/list": 401 Unauthorized

I tried HisashiHasebe/hasebetest-nuxt-auth@b16500b , And local build on cloned your repository.

@HisashiHasebe
Copy link
Contributor Author

@VortexExpansion
I have one more concern.

The front-nuxt-auth has a feature that allows changing the site key from front-end.
I think the request destination for ofetch entered in api/index.js cannot be changed from the front side, Is it OK?

@VortexExpansion
Copy link
Contributor

@HisashiHasebe san
Sorry for confusion.
I meant please check the deployed site, not this PR.

https://dev-nuxt-auth.g.kuroco-front.app/member/detail/141/

This has been hotfixed so I request you to confirm if that behaviour is sufficient so that I can close this PR.

@HisashiHasebe
Copy link
Contributor Author

@VortexExpansion
Oh, Sorry.

https://dev-nuxt-auth.g.kuroco-front.app/member/detail/141/ is OK now for Abe san update(74472c5 , 8af94d8)

I will close this issue.

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 a pull request may close this issue.

2 participants