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

[WIP] Nuxt3 project #1

Merged
merged 92 commits into from
Nov 13, 2023
Merged

Conversation

VortexExpansion
Copy link
Contributor

@VortexExpansion VortexExpansion commented Sep 8, 2023

Reviewer

@yabe-diverta @terasim

Related

https://github.com/diverta/front_kuroco_document_site/issues/348
https://github.com/diverta/Kuroco-opendev/issues/4907

Checklist

  • Create front-end repository
  • Kuroco backend creation
  • Import the latest UI using sass
  • Copying Kuroco content and other data
  • Refine the front end with Nuxt3
  • Refactor typescript to javascript
  • Implement auth using auth.nuxtjs
  • Refactor options API to compositions API

@yabe-diverta
Copy link
Collaborator

yabe-diverta commented Sep 15, 2023

@VortexExpansion

  • add .env.example to demonstrate clients how to setup env var
  • make sure it works with FullStaticGeneration, especially for dynamic routes (like [id] route)
  • deploy it on Kuroco front, provide GithubActions workflow (if you are not familiar with it, please notice me)
  • remove warning on running dev, like:
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.

Recommendation: math.div($value * 10, 2) or calc($value * 10 / 2)

More info and automated migrator: https://sass-lang.com/d/slash-div

   ╷
79 │   .u-mt-#{$value * 10/2} {
   │           ^^^^^^^^^^^^^
   ╵
    assets/scss/object/utility/_utility.scss 79:11  @import
    assets/scss/style.scss 26:9                     root stylesheet

@yabe-diverta
Copy link
Collaborator

yabe-diverta commented Oct 26, 2023

@HisashiHasebe

コンポーネント化する時のルールというか、指標みたいなのってありますか?

最低限同じコードを生成するものをコンポーネント化しています。

コンポーネント化するのは良いのですが、ユーザー側で再利用しやすい("ユーザー側で編集する前提"って観点で)とか読みやすいとかって場合に実施して欲しいです。

了解しました。
- AppFooter
- AppHeader
- ui > AlertSuccess
- ui > AlertError
- ui > SubmitButton

ぐらいは残しても良いかなと思っているのですが、
お手数ですが他にいる(orいらない)ものがあるかご確認お願いできますでしょうか。
https://github.com/VortexExpansion/front_nuxt_corporate/tree/refactored-javascript/components

失礼しました、ちょっと見直してみますね。

@KentaKatoh
Copy link
Member

@yabe-diverta コンポーネント設計は阿部さんがやった方がいいと思うので、投げないでください。

@yabe-diverta
Copy link
Collaborator

yabe-diverta commented Oct 26, 2023

@yabe-diverta
Copy link
Collaborator

yabe-diverta commented Oct 27, 2023

メモ:

useFetch()を使う時、

  • arg1のURLを相対パスにする(/some/api)
  • arg2のオプションにbaseURLを入れる

ケースでは、
SSRのデータ取得時にresponseがjsonに自動変換されない(textになる)

[email protected]で再現するが、
たまたま4d前に修正されている模様なので、近々修正されると思われる。

修正がバージョンに入るまではbaseURLの使用をやめて、
arg1のURLを絶対パスにして対応する。
修正バージョンが入り次第nuxtをアップデート

@yabe-diverta
Copy link
Collaborator

メモ:
@sidebar/nuxt-auth cookie feat req

@HisashiHasebe
Copy link
Collaborator

HisashiHasebe commented Nov 3, 2023

【追加】

@HisashiHasebe
Copy link
Collaborator

HisashiHasebe commented Nov 3, 2023

【メモ】
コピー元サイト作成準備のため、バックエンド側のコンテンツ定義やカスタム処理の名前などを英語にしました。
フロントへの影響はないはずです。

@yabe-diverta
Copy link
Collaborator

yabe-diverta commented Nov 7, 2023

  • partitioned cookie の設定, 動作確認

@HisashiHasebe
Copy link
Collaborator

HisashiHasebe commented Nov 8, 2023

【追加】

@yabe-diverta
Copy link
Collaborator

@HisashiHasebe @takel-shiba
レビュー一旦Doneとなったと確認できたと思っているのでマージしたいのですが、
阿部だとマージボタンが表示されないみたいです。
お手数ですがマージお願いしてもよいでしょうか。

@yabe-diverta
Copy link
Collaborator

@VortexExpansion
Sorry I forgot mentioning above to you.
Please merge it if you can?

@VortexExpansion
Copy link
Contributor Author

@yabe-diverta san
Sorry, I don't have the write access to diverta:main too.

I think only @takel-shiba san has the merge authority. Please review.

@takel-shiba takel-shiba merged commit 91aefeb into diverta:main Nov 13, 2023
@takel-shiba
Copy link
Collaborator

@VortexExpansion @yabe-diverta @HisashiHasebe
I have merged now!

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.

5 participants