Add admin panel and user roles #189
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add with_env.sh as dev.sh replacement This new script can load env vars from any path, supplied with the optional --env option (still defaults to .env) and accepts any arbitrary command after that: `mix phx.new`, `iex -S mix phx.server`, `mix test`. This provides support for having multiple `.env*` files. One for dev, one for test, for example. The previous setup wasn't picking up the right Postgres connection string for tests and was creating problems. * Fix assorted compilation warnings * Make :name and :started_at required in event changesets * Refactor get_event!/2 to take ID and UUID * Make event ordering more consistent * Fix spelling of leaded_by with led_by * Refactor get_managed_event!/3 While testing I found that this did not return `Ecto.NoResultsError` when no events were found. This new version does and it also uses a single database query. * Make get_different_event_with_code/2 private * Enable youtube.com pattern for YouTube embeds * Refactor duplicate_event/2 When duplicating an event without a presentation, an error was raised. I attempted to fix that and ended up cleaning up the implementation a bit. I also discovered issue #178 was raised, so I fixed the implementation to skip copying `vote_count` for poll options. * Add activity leader test fixture * Make DataCase db connection check out conditional * Expand event tests
- Updated `assign_role/2` function to accept both role names and role structs in `Claper.Accounts`. - Modified `export_users_to_csv/1` to include user names and roles in the CSV export. - Enhanced `export_events_to_csv/1` to include event descriptions and statuses in the CSV export. - Adjusted CSV headers and fields accordingly for better clarity. - Removed unused user attributes in role tests to streamline setup. - Updated tests for CSV export to reflect changes in data structure and content. - Removed outdated tests for admin routes, user controller, OIDC provider controller, and admin form validation. - Deleted tests for admin required plug and shared view components as part of cleanup.
…ration changeset to include role_id
… user_id requirement from event changeset
# Conflicts: # CHANGELOG.md # config/runtime.exs # lib/claper/accounts/user.ex # lib/claper_web/live/user_settings_live/show.html.heex # priv/gettext/de/LC_MESSAGES/default.po # priv/gettext/default.pot # priv/gettext/en/LC_MESSAGES/default.po # priv/gettext/es/LC_MESSAGES/default.po # priv/gettext/fr/LC_MESSAGES/default.po # priv/gettext/it/LC_MESSAGES/default.po # priv/gettext/nl/LC_MESSAGES/default.po
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.