wri / wri_sites

WRI profile
2 stars 2 forks source link

Modals: Full gating of research downloads #226

Closed jordankaiser closed 11 months ago

jordankaiser commented 11 months ago

What issue(s) does this solve?

Styles a content gate modal as described in this issue. Thanks!

What is the new behavior?

Profile requirements:

Site-level pull requests for testing. Only merge when these PRs are approved:

Create or update any site-level pull requests following the documentation

Checked on develop (TA to do)

jordankaiser commented 11 months ago

@mariacha Ran into the the attached error while trying to look at these changes. Let me know if you have any thoughts on how to resolve and I can get these changes going, thanks!

The wri_sites repo is on branch issue-301-restarted. I merged main into issue-301-restarted. The wriflagship repo is on branch develop.

Screenshot 2023-10-16 at 9 47 51 AM

mariacha commented 11 months ago

Ahh, @jordankaiser Start from this branch on wriflagship: issue-301-modals (that's this PR https://github.com/wri/wriflagship/pull/1125) -- I bet there's some config in there that's important.

jordankaiser commented 11 months ago

@mariacha I believe I've got this all addressed, thanks!