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

First Test Frame #2

Merged
merged 6 commits into from
Feb 3, 2024
Merged

First Test Frame #2

merged 6 commits into from
Feb 3, 2024

Conversation

hurley87
Copy link
Collaborator

@hurley87 hurley87 commented Feb 3, 2024

No description provided.

const inter = Inter({ subsets: ['latin'] });

export const metadata: Metadata = {
title: 'Create Next App',
Copy link
Owner

Choose a reason for hiding this comment

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

Can this metadata be edited?

console.log('buttonIndex: ', buttonIndex);

if (buttonIndex === 2) {
return NextResponse.redirect('https://frameception.vercel.app', {
Copy link
Owner

Choose a reason for hiding this comment

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

probably use "NEXT_PUBLIC_URL" here?

import { FrameRequest, getFrameHtmlResponse } from '@coinbase/onchainkit';
import { NextRequest, NextResponse } from 'next/server';

const NEXT_PUBLIC_URL = 'https://frameception.vercel.app';
Copy link
Owner

Choose a reason for hiding this comment

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

Should this be an env var?

Copy link
Owner

@smallbraingames smallbraingames left a comment

Choose a reason for hiding this comment

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

Can you change this commit name also LOL (the "first-frame" one)

Copy link
Owner

@smallbraingames smallbraingames left a comment

Choose a reason for hiding this comment

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

preapproving

@hurley87 hurley87 merged commit 8c1cacd into main Feb 3, 2024
@smallbraingames smallbraingames deleted the firstframe branch February 3, 2024 22:23
@smallbraingames smallbraingames restored the firstframe branch February 3, 2024 22:23
@smallbraingames smallbraingames deleted the firstframe branch February 4, 2024 04:55
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.

2 participants