ThreeSixtyGiving / registry-classic

Registry for 360Giving files that were published
Other
0 stars 0 forks source link

Prerequisites for registry deprecation #134

Closed codemacabre closed 1 year ago

codemacabre commented 1 year ago

Summary

Fixes https://github.com/ThreeSixtyGiving/registry/issues/84

Fixes https://github.com/ThreeSixtyGiving/registry/issues/107

Fixes https://github.com/ThreeSixtyGiving/registry/issues/75

Fixes https://github.com/ThreeSixtyGiving/registry/issues/118

Fixes https://github.com/ThreeSixtyGiving/registry/issues/120

Fixes https://github.com/ThreeSixtyGiving/registry/issues/121

Fixes https://github.com/ThreeSixtyGiving/registry/issues/111

Fixes https://github.com/ThreeSixtyGiving/registry/issues/138

Fixes https://github.com/ThreeSixtyGiving/registry/issues/122

michaelwood commented 1 year ago

Test looks like it's failing because we're getting blocked by twitter via a 403 http status code. I think it's fine to put something like:

for link in links:
+   # twitter blocking requests
+    if "twitter" in link.get_attribute("href"):
+        continue

in https://github.com/ThreeSixtyGiving/registry/blob/dashboard/tests/test_functional.py#L139 to skip over that one.

michaelwood commented 1 year ago

Thanks for all this work. Work combined into #141 . Future improvements and developments can now be done in a branch off of "dashbaord" branch.