Skip to content

Commit

Permalink
Drop should publish field from ReservedList entity (#2369)
Browse files Browse the repository at this point in the history
* Remove the shouldPublish field from the reservedLIst entity

* Add generated schema file
  • Loading branch information
sarahcaseybot authored Mar 14, 2024
1 parent 6e57d93 commit 5f9c7de
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,6 @@ public final class ReservedList
*/
@Insignificant @Transient Map<String, ReservedListEntry> reservedListMap;

// TODO(b/321053918): Remove this field once the column is nullable
@Column(nullable = true)
boolean shouldPublish = true;

@PreRemove
void preRemove() {
tm().query("DELETE FROM ReservedEntry WHERE revision_id = :revisionId")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ void testRun_withWildcard() {
Optional.of("*"),
Optional.empty(),
Optional.empty(),
"^name\\s+.*shouldPublish.*",
"^name\\s+.*",
"^-+\\s+-+",
"^xn--q9jyb4c-private\\s+.*",
"^xn--q9jyb4c-published\\s+.*");
Expand Down
1 change: 0 additions & 1 deletion db/src/main/resources/sql/schema/db-schema.sql.generated
Original file line number Diff line number Diff line change
Expand Up @@ -693,7 +693,6 @@
revision_id bigserial not null,
creation_timestamp timestamptz,
name text not null,
should_publish boolean,
primary key (revision_id)
);

Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# This file defines properties used by the Gradle build. It must be kept in
# This file defines properties used by the gradle build. It must be kept in
# sync with config/nom_build.py.
#
# To regenerate, run ./nom_build --generate-gradle-properties
Expand Down

0 comments on commit 5f9c7de

Please sign in to comment.