cielf commented 10 months ago

Summary

Admin paths currently require a organization_id, which is the short_name. Unless actually dealing with an organization, they shouldn't. Also address the mismatch between the parameter name and what is used.

Why fix

Confusion. Current solution is kludgey (having to hackishly send a faux short name to get to admin things)

Details

Admin paths currently require a organization_id, which is the short_name.
Unless actually dealing with an organization, they shouldn't.
Also address the mismatch between the parameter name (organization_id) and what is used (short_name).

Notes

Origin of issue

See discussion on https://github.com/rubyforgood/human-essentials/pull/3664

Criteria for completion

[ ] The admin paths no longer require an organization id when they aren't dealing with an org
[ ] The parameter name used to identify organizations and what is passed match
[ ] It doesn't break anything else -- ALL tests work

github-actions[bot] commented 9 months ago

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] commented 8 months ago