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

NGSTACK-883 add missing icons and replace base64 with svg icons #199

Open
wants to merge 65 commits into
base: netgen-org-site
Choose a base branch
from

Conversation

TTrakostanec
Copy link
Contributor

@TTrakostanec TTrakostanec commented May 8, 2024

For some reason it didn't add flatpickr (which was added for this task https://netgen.atlassian.net/browse/NGSTACK-873) so I had to add it again

iherak and others added 30 commits May 3, 2024 12:45
There are changes here committed that must be removed before making
a PR to media-site!
PetraMiseta and others added 23 commits May 6, 2024 11:11
…yling

NGSTACK-876 add scss files for notification module
NGSTACK-853 remove unecessary changes

NGSTACK-853 add empty line at the end of the file
@TTrakostanec TTrakostanec requested a review from Ljudevit May 8, 2024 10:38
@@ -74,6 +74,7 @@
"@netgen/javascript-cookie-control": "^0.0.8",
"@popperjs/core": "^2.11.6",
"bootstrap": "^5.3.3",
"flatpickr": "^4.6.13",
Copy link
Member

Choose a reason for hiding this comment

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

Again wrong rebase ? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah...I don't know what happened, but I had to add it again

@emodric
Copy link
Member

emodric commented May 13, 2024

@Ljudevit Please review!

@emodric
Copy link
Member

emodric commented May 13, 2024

@TTrakostanec Before merging, the icons should go to assets/backoffice/icons.

@emodric emodric force-pushed the netgen-org-site branch 2 times, most recently from 392e5f2 to 8cc0aaf Compare May 24, 2024 11:02
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