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

[17]APIの実装周り修正 #24

Open
wants to merge 8 commits into
base: dev-akidon
Choose a base branch
from

Conversation

marlion01
Copy link
Contributor

Issues

Closes #17

概要

API関連を実装

説明(懸念点や注意点)

git fetchの際は
flutter pub run build_runnner build
でクラスを実装する必要あり

キャプチャ

@akidon0000 akidon0000 self-requested a review January 5, 2024 05:28
@@ -0,0 +1,17 @@
import 'package:freezed_annotation/freezed_annotation.dart';
Copy link
Member

Choose a reason for hiding this comment

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

niche

adminAPIもadmin_apiやね

@freezed
class PrItems with _$PrItems{
const factory PrItems({
required String prItems
Copy link
Member

Choose a reason for hiding this comment

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

comment

https://github.com/tokudai0000/tokumemo_resource/blob/main/api/v1/ad_items.json

id
clientName
imageUrlStr
targetUrlStr
imageDescription

が必要そう

const factory CurrentTermVersion({required String currentTermVersion})= _CurrentTermVersion;
// json形式で受け取るためのコードを生成するために記述
factory CurrentTermVersion.fromJson(Map<String,dynamic> json)=>_$CurrentTermVersionFromJson(json);

Copy link
Member

Choose a reason for hiding this comment

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

niche

この行を削除してくれると。。。💦

@@ -0,0 +1,13 @@
// ignore: file_names
Copy link
Member

Choose a reason for hiding this comment

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

ask-badge

このコメントは何表してる??

@akidon0000
Copy link
Member

comment

https://zenn.dev/koji_1009/articles/e769febac0a8c5

.g.dartや.freezed.dartはGitにあげてよさそうやね

@akidon0000 akidon0000 assigned marlion01 and unassigned akidon0000 Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants