condo icon indicating copy to clipboard operation
condo copied to clipboard

Use `antd-phone-input` over `react-phone-input-2`

Open ArtyomVancyan opened this issue 2 years ago • 2 comments

The antd-phone-input is designed to use with Ant Design and is well maintained by me and I highly recommend using it. It is compatible with the version of antd you are using. I tried to open a PR with the necessary changes but could not run the workspace on the development environment (it uses huge storage and has issues with migrations or your Getting started section of the README need to be updated).

Thanks for your attention.

ArtyomVancyan avatar Sep 28 '23 09:09 ArtyomVancyan

Can you contribute this changes?

pahaz avatar Mar 31 '24 15:03 pahaz

Yes, I do. But as I mentioned in the description, I need help with setting up the development environment and running it locally. I followed the steps in README and the development guide. Even had to change the usage of the class-based components to pass the build. The yarn workspace @app/condo dev logs the following:

Loaded env from /home/artyom/Python/condo/apps/condo/.env
warn  - Detected next.config.js, no exported configuration found. https://err.sh/vercel/next.js/empty-configuration
(node:167284) Warning: Accessing non-existent property 'default' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
(node:167284) Warning: Accessing non-existent property 'target' of module exports inside circular dependency
(node:167284) Warning: Accessing non-existent property 'amp' of module exports inside circular dependency
(node:167284) Warning: Accessing non-existent property 'experimental' of module exports inside circular dependency

P.S. I tried with different Node versions (14, 16, 18), and the result is the same.

ArtyomVancyan avatar Apr 01 '24 13:04 ArtyomVancyan

Seems like this issue is stale. Setting up repo should be far more simple now

Closing the issue

toplenboren avatar Oct 24 '24 18:10 toplenboren