Skip to content

Conversation

@huanphan-tma
Copy link

@huanphan-tma huanphan-tma commented May 29, 2024

  • Ticket: GRDM-40446
  • Feature flag: n/a

Purpose

Implement 2.1.1.認証属性によるログインアクセス制御 function

Summary of Changes

  • Add login availability not available message UI to the home page.
  • Add login availability not available message translations.

Side Effects

QA Notes

Copy link
Contributor

@yacchin1205 yacchin1205 left a comment

Choose a reason for hiding this comment

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

変更は問題ないかと思いましたが、問題発生時に後で追跡できるように、

  • Ticket: の欄へのTicket番号の追加と、
  • Titleに変更(主に意図)のわかる記述の記載をお願いします。(できれば、commitも、Add source codeはほぼ内容がないのと一緒なので...直していただくのが理想的とは思うのですが...それは次回以降でも良いのではと思いました。)

参考: #73

@huanphan-tma huanphan-tma changed the title feature/grdm-202401-step2/login-access-control-management Add login access control function based on authentication attributes and email address Jun 12, 2024
@hidefumi-moritani
Copy link

コメントありがとうございます。

変更は問題ないかと思いましたが、問題発生時に後で追跡できるように、

  • Ticket: の欄へのTicket番号の追加と、
  • Titleに変更(主に意図)のわかる記述の記載をお願いします。

上記の修正を実施いたしました。

(できれば、commitも、Add source codeはほぼ内容がないのと一緒なので...直していただくのが理想的とは思うのですが...それは次回以降でも良いのではと思いました。)

参考: #73

今回の開発で commit を修正するためには --force-with-lease オプションを利用する必要があると思いますので、
次回以降の対応とさせてください。

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