Skip to content

change is_public in groups table to tinyint#317

Open
rimas-kudelis wants to merge 9 commits into
masterfrom
runout-at-groups-boolfield
Open

change is_public in groups table to tinyint#317
rimas-kudelis wants to merge 9 commits into
masterfrom
runout-at-groups-boolfield

Conversation

@rimas-kudelis

Copy link
Copy Markdown
Collaborator

Replaces #257

runout-at and others added 8 commits May 5, 2018 12:37
merge vexim2 to runout
To have consistent fieldtypes in vexim
To have consistent fieldtypes in vexim
To have consistent fieldtypes in vexim
To have consistent fieldtypes in vexim
To have consistent fieldtypes in vexim
To have consistent fieldtypes in vexim
Fix a typo

@runout-at runout-at left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you very much! Looks good to me!

@rimas-kudelis

Copy link
Copy Markdown
Collaborator Author

Do we have anyone to test with PostgreSQL?

@rimas-kudelis

Copy link
Copy Markdown
Collaborator Author

Oh I forgot. We will also need a migration SQL file for this. It should first alter the data, then change column type.

Until we decide to make a release, I guess it could be called descriptively by what it does. Before tagging new version, we'd merge all these changes into one file.

@rimas-kudelis

rimas-kudelis commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator Author

Alright, so we need a migration here.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants