MAPC / massbuilds

Crowdsourcing authoritative info on recent & ongoing developments in Massachusetts.

https://www.massbuilds.com/

2 stars 2 forks source link

Mjz/309 fix permissions issue #310

Closed mzagaja closed 3 years ago

mzagaja commented 3 years ago

Resolves #309.

Why is this change necessary?

Municipal users were not able to add developments despite the fact they should be able to do so.

How does it address the issue?

We update the strings from Pelias to be UPCASE before doing the comparison so it works going forward.

What side effects does it have?

We also add fixes for previously undiscovered setup issues related to foreign data wrappers that were necessary to resolve this bug from a fresh setup of the develop branch.

mzagaja commented 3 years ago

@atomtay That's a good catch. I should be able to fix it by adding a safe navigation operator. Will give that a try and then you can test again.

atomtay commented 3 years ago

Re-factor looks good and works for both Cambridge and Acton. Ready to merge!