Skip to content

Conversation

@icarusinthewings
Copy link
Contributor

The title says it all.
Tested it, and it seems to work.
If you have any questions, just let me know :)

@icarusinthewings
Copy link
Contributor Author

@JaydenDev Take a look at this.

@jdev082
Copy link
Collaborator

jdev082 commented Dec 19, 2021

so what exactly is bump npm packaging.

@icarusinthewings
Copy link
Contributor Author

I don't know, it said that the package-lock.json was made with an old version and it resolved that.

@icarusinthewings
Copy link
Contributor Author

I could .gitignore that and remake the pull if you would like.

@jdev082
Copy link
Collaborator

jdev082 commented Dec 19, 2021

oh its all good, merging btw I got font awesome icons working and replacd the text on the buttons to the font awesome icons
and also moved em to the top

@jdev082 jdev082 merged commit 0fb281a into CatalystDevOrg:master Dec 19, 2021
@jdev082
Copy link
Collaborator

jdev082 commented Dec 19, 2021

i forgot to git push my commits before merging, reverting.

@icarusinthewings
Copy link
Contributor Author

Do I need to pull from upstream?

@icarusinthewings icarusinthewings mentioned this pull request Dec 19, 2021
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.

2 participants