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.

Sign up for GitHub

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

Jump to bottom

Release pro plan #287

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Release pro plan #287

merged 2 commits into from
Jan 15, 2025

Conversation

satococoa
Copy link
Contributor

@satococoa satococoa commented Jan 6, 2025 •
edited
Loading

Summary

This PR activates the Pro plan and a team creation.
Also, the agent time included in the Pro plan is changed from 120 to 60 minutes.

Related Issue

Changes

  • Activate proTeamPlanFlag by default
  • Activate teamCreationFlag by default
  • Change AGENT_TIME_CHARGE_LIMIT_MINUTES for PRO

Testing

  • Create a new Giselle account
  • Upgrade the default team to Pro
  • Included Agent time with Pro plan is 60
  • Create and run Agent
  • Create only one Free team
  • Create a Pro team
  • Add a member to a Pro team
  • Remove a member from a Pro team
  • Change a member’s role in a Pro team
  • Ensure that you cannot add a member to a Free team

Other Information

@satococoa satococoa self-assigned this Jan 6, 2025
@vercel Vercel
Copy link

vercel bot commented Jan 6, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
giselle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 14, 2025 1:17am

@satococoa satococoa marked this pull request as ready for review January 14, 2025 04:16
@satococoa satococoa changed the title [DO NOT MERGE UNTIL Release date] Release pro plan Release pro plan Jan 14, 2025
shige
shige approved these changes Jan 14, 2025
View reviewed changes
Copy link
Member

@shige shige left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@satococoa satococoa merged commit e8f7252 into main Jan 15, 2025
9 checks passed
@satococoa satococoa deleted the release-pro-plan branch January 15, 2025 00:32
@shige shige mentioned this pull request Feb 13, 2025
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toyamarinyon toyamarinyon toyamarinyon approved these changes

@shige shige shige approved these changes

Assignees

@satococoa satococoa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@satococoa @toyamarinyon @shige