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

Update audits.stub #992

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update audits.stub #992

wants to merge 1 commit into from

Conversation

MahammadBaghirzada
Copy link

enhanced auditable_id to be string instead of int

@MahammadBaghirzada
Update audits.stub
36330ee 
enhanced auditable_id to be string instead of int
willpower232
willpower232 requested changes Jan 13, 2025
View reviewed changes
Copy link
Contributor

@willpower232 willpower232 left a comment

Choose a reason for hiding this comment

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

If you need it to be a string for your application then you can publish the migration and edit as you need however for the vast majority of users, unsignedBigInteger is correct to match the default of $table->id() in regular migrations.

@parallels999
Copy link

Maybe just a comment like string for UUID

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@willpower232 willpower232 willpower232 requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MahammadBaghirzada @parallels999 @willpower232