Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove active admin #611

Merged
merged 9 commits into from
May 26, 2022
Merged

Remove active admin #611

merged 9 commits into from
May 26, 2022

Conversation

baarkerlounger
Copy link
Contributor

@baarkerlounger baarkerlounger commented May 25, 2022

Remove ActiveAdmin and AdminUser class since we're building support journeys into the app itself and are no longer expecting to use this.

Simplifies our dependency graph, and Devise user model.

@baarkerlounger baarkerlounger marked this pull request as ready for review May 26, 2022 15:04
Copy link
Contributor

@dushan-madetech dushan-madetech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of deletions so giving this an approval. As long as the tests/linting passes 🚢 it

@baarkerlounger baarkerlounger merged commit 402eddc into main May 26, 2022
@baarkerlounger baarkerlounger deleted the remove_active_admin branch May 26, 2022 16:02
baarkerlounger added a commit that referenced this pull request May 27, 2022
* Remove stuff

* Remove ActiveAdmin from Gem file

* Remove routes and table

* Rubocop

* Remove active admin from webpack config

* Remove JQuery from webpack

* Remove remaining spec references

* Remove js packages

* Schema
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants