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

Various improvements and fixes #139

Merged
merged 4 commits into from
Jan 28, 2024
Merged

Various improvements and fixes #139

merged 4 commits into from
Jan 28, 2024

Conversation

JordiVanOs
Copy link
Member

Adds link to the admin panel for admins
Adds link for division contacts to admin panel
Adds restrict (read only) access to members for division contacts within their division
Improves styling
Updates logo
Improves mobile responsiveness
Removes legacy division overview (/groep/)

@JordiVanOs JordiVanOs self-assigned this Dec 5, 2023
@Nowa-Ammerlaan
Copy link
Member

I'm getting:

An exception has been thrown during the rendering of a template ("Could not find the entrypoints file from Webpack: the file "/var/www/project/public/assets/entrypoints.json" does not exist.").

@Xesau
Copy link
Contributor

Xesau commented Dec 7, 2023

Run yarn dev of yarn build to generate the entrypoints.json file

Copy link
Member

@Nowa-Ammerlaan Nowa-Ammerlaan left a comment

Choose a reason for hiding this comment

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

That fixed it.

Some minor comments:

  • please update readme with new login info for admindebaas
  • if this yarn build is going to be always required to get this up and running, please add this step to readme as well.
  • and finally:
    image
    I am personally not fond of this blue in our red colour scheme. Can we make this black instead?

@JordiVanOs JordiVanOs changed the title Adds admin link Various improvements and fixes Jan 28, 2024
@Nowa-Ammerlaan Nowa-Ammerlaan merged commit f99328b into main Jan 28, 2024
1 check passed
@Nowa-Ammerlaan Nowa-Ammerlaan deleted the feature/add-admin-link branch January 28, 2024 18:58
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.

5 participants