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

feat: Implement new link field #5373

Merged
merged 10 commits into from
Nov 10, 2020

Conversation

iamareebjamal
Copy link
Member

@iamareebjamal iamareebjamal commented Oct 24, 2020

Fixes #5116

image

@vercel
Copy link

vercel bot commented Oct 24, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/mn9rbh2yp
✅ Preview: https://open-event-frontend-git-link-field.eventyay.vercel.app

return splitted[1] || splitted[0];
}

@action
Copy link
Member

Choose a reason for hiding this comment

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

Codacy found an issue: Missing return type on function.

Copy link
Member

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 3
           

Complexity increasing per file
==============================
- app/components/widgets/forms/link-field.ts  2
- tests/integration/components/widgets/forms/link-field-test.ts  1
         

See the complete overview on Codacy

}

@action
setValue(event: any) {
Copy link
Member

Choose a reason for hiding this comment

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

return this.prefix + this.fixedValue;
}

get fixedValue() {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy found an issue: Missing return type on function.

@codecov
Copy link

codecov bot commented Nov 10, 2020

Codecov Report

Merging #5373 (0634905) into development (d21b442) will increase coverage by 0.37%.
The diff coverage is 65.78%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #5373      +/-   ##
===============================================
+ Coverage        23.32%   23.69%   +0.37%     
===============================================
  Files              494      497       +3     
  Lines             5226     5254      +28     
  Branches            39       44       +5     
===============================================
+ Hits              1219     1245      +26     
- Misses            4001     4003       +2     
  Partials             6        6              
Impacted Files Coverage Δ
app/components/forms/wizard/other-details-step.js 0.00% <0.00%> (ø)
app/components/widgets/forms/link-input.js 33.33% <0.00%> (ø)
app/models/social-link.js 0.00% <0.00%> (ø)
app/utils/computed-helpers.js 19.56% <0.00%> (-1.72%) ⬇️
app/components/widgets/forms/link-field.ts 75.00% <75.00%> (ø)
app/components/widgets/forms/social-link-field.ts 75.00% <75.00%> (ø)
app/utils/dictionary/social-media.ts 100.00% <100.00%> (ø)
app/components/tabbed-navigation.js 33.33% <0.00%> (-20.00%) ⬇️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d21b442...187886c. Read the comment docs.

@mariobehling
Copy link
Member

Nice!

@iamareebjamal
Copy link
Member Author

Done

@iamareebjamal iamareebjamal merged commit 2296ea7 into fossasia:development Nov 10, 2020
@iamareebjamal iamareebjamal deleted the link-field branch November 10, 2020 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wizard Step 2: Cannot change first option of social links (website)
3 participants