Skip to content

Commit

Permalink
Issue GENI-NSF#164: DB schema change to allow email to be null on acc…
Browse files Browse the repository at this point in the history
…ount requests (for tutorials)
  • Loading branch information
ahelsing committed Jul 22, 2016
1 parent 75d4c7b commit 70832b1
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@
```
psql -U accreq -h localhost accreq < /usr/share/geni-ar/db/postgresql/update-3.sql
```
* Allow null `email` in `idp_account_request` (for tutorials):
```
psql -U accreq -h localhost accreq < /usr/share/geni-ar/db/postgresql/update-4.sql
```

# [Release 1.8](https://github.com/GENI-NSF/geni-ar/milestones/1.8)

Expand Down
1 change: 1 addition & 0 deletions Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ nobase_dist_pkgdata_DATA = \
db/postgresql/update-1.sql \
db/postgresql/update-2.sql \
db/postgresql/update-3.sql \
db/postgresql/update-4.sql \
etc/confirm-email.txt \
etc/leads-email.txt \
etc/notification-email.txt \
Expand Down
2 changes: 1 addition & 1 deletion db/postgresql/schema.sql
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ CREATE TABLE idp_account_request (
id SERIAL PRIMARY KEY,
first_name VARCHAR NOT NULL,
last_name VARCHAR NOT NULL,
email VARCHAR NOT NULL,
email VARCHAR,
username_requested VARCHAR NOT NULL,
phone VARCHAR NOT NULL,
password_hash VARCHAR NOT NULL,
Expand Down
2 changes: 2 additions & 0 deletions db/postgresql/update-4.sql
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-- Email is now optional for tutorial accounts
ALTER TABLE idp_account_request ALTER COLUMN email DROP NOT NULL;
1 change: 1 addition & 0 deletions geni-ar.spec
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ rm -rf $RPM_BUILD_ROOT
%{_datadir}/%{name}/db/postgresql/update-1.sql
%{_datadir}/%{name}/db/postgresql/update-2.sql
%{_datadir}/%{name}/db/postgresql/update-3.sql
%{_datadir}/%{name}/db/postgresql/update-4.sql
%{_datadir}/%{name}/etc/confirm-email.txt
%{_datadir}/%{name}/etc/leads-email.txt
%{_datadir}/%{name}/etc/notification-email.txt
Expand Down

0 comments on commit 70832b1

Please sign in to comment.