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

Nimi Themes Naming improvement #297

Closed
wants to merge 1 commit into from

Conversation

b8zeek
Copy link
Contributor

@b8zeek b8zeek commented Jan 27, 2023

Summary

This pull request updates naming of Nimi themes in Nimi App as per the changes made on Nimi Card repository in this pull request.

Together with the above mentioned pull request, resolves #281.

P.S. @adamazad, after merging, during the new release improve Nimi Card's dependency.

🍀

@vercel
Copy link

vercel bot commented Jan 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nimi-io-app-development ❌ Failed (Inspect) Jan 27, 2023 at 1:26PM (UTC)
nimi-io-app-production ❌ Failed (Inspect) Jan 27, 2023 at 1:26PM (UTC)

@b8zeek b8zeek requested review from Mi-Lan and adamazad January 27, 2023 13:26
@Mi-Lan
Copy link
Collaborator

Mi-Lan commented Jan 27, 2023

@adamazad also would be good to make sure that themes on older profiles are backward compatible

@b8zeek
Copy link
Contributor Author

b8zeek commented Jan 29, 2023

@adamazad & @Mi-Lan, please review this pull request. Thanks.

@Mi-Lan
Copy link
Collaborator

Mi-Lan commented Feb 1, 2023

I would merge it but would be good to have mapping of old values to new one on the server so that previously published profiles would work as intended when we repopulate data...I guess thats on @adamazad

@adamazad adamazad force-pushed the development branch 2 times, most recently from 0ee4978 to be7fa40 Compare February 2, 2023 00:29
@adamazad adamazad closed this Apr 10, 2023
@adamazad adamazad deleted the themes-naming-improvement branch May 11, 2023 11:32
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.

Address Themes
3 participants