Skip to content

Auto Generate Custom URLs#9

Merged
xprilion merged 17 commits intoxprilion:masterfrom
luckykumarirai:auto-generate-url
Jan 5, 2021
Merged

Auto Generate Custom URLs#9
xprilion merged 17 commits intoxprilion:masterfrom
luckykumarirai:auto-generate-url

Conversation

@luckykumarirai
Copy link
Copy Markdown
Contributor

@luckykumarirai luckykumarirai commented Dec 12, 2020

What is the current behavior?

Every time needs to enter a custom URL.

What is the new behavior?

Whenever we do not want to enter a custom URL by click on the button auto-generate 5-10 character code as a custom URL.
if we think need to give our own custom URL then we can enter both functionality present.

Review link:

https://fireshort-e4651.web.app/admin

email:- shorturl@gmail.com
Password:- Url@123

close #5

@ashuvssut
Copy link
Copy Markdown
Collaborator

hello! please provide new test 'authentication credentials' from firebase to test your changes.

@xprilion
Copy link
Copy Markdown
Owner

@luckykumarirai thanks for the PR! I have tested that while creating new links its easy to generate random characters, however I am unable to change the same when editing the links already created. Please fix.

@xprilion xprilion self-requested a review December 23, 2020 04:55
Copy link
Copy Markdown
Owner

@xprilion xprilion left a comment

Choose a reason for hiding this comment

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

Fix generating random characters when editing links

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

Now I have fixed generating random characters when editing links problem.

@luckykumarirai luckykumarirai changed the title Auto Generate Custom Url Auto Generate Custom URLS Jan 4, 2021
@luckykumarirai luckykumarirai changed the title Auto Generate Custom URLS Auto Generate Custom URLs Jan 4, 2021
@xprilion xprilion merged commit cfa55b9 into xprilion:master Jan 5, 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.

Add auto short code generator, rather than always entering custom URLs

3 participants