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

Change Icon type to String and add Application type field to in AppSchema #1211

Closed
wants to merge 2 commits into from
Closed

Change Icon type to String and add Application type field to in AppSchema #1211

wants to merge 2 commits into from

Conversation

medchedli
Copy link
Member

@medchedli medchedli commented Oct 19, 2023

Replaced the application Icon type from Buffer to String and added application type (which can holds either "link" or "embedded" as values)

@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (69e71eb) 86.75% compared to head (c021fcb) 86.76%.

❗ Current head c021fcb differs from pull request most recent head 47e908d. Consider uploading reports for the commit 47e908d to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1211      +/-   ##
==========================================
+ Coverage   86.75%   86.76%   +0.01%     
==========================================
  Files          89       89              
  Lines        6024     6024              
==========================================
+ Hits         5226     5227       +1     
+ Misses        798      797       -1     
Files Coverage Δ
src/model/apps.js 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@medchedli medchedli marked this pull request as ready for review October 23, 2023 09:04
@medchedli medchedli changed the title Change Icon type to String in AppSchema Change Icon type to String and Application type field to in AppSchema Oct 25, 2023
@medchedli medchedli changed the title Change Icon type to String and Application type field to in AppSchema Change Icon type to String and add Application type field to in AppSchema Oct 25, 2023
@@ -30,4 +32,4 @@ const AppSchema = new Schema({
})

export const AppModelAPI = connectionAPI.model('App', AppSchema)
export const AppModel = connectionDefault.model('App', AppSchema)
export const AppModel = connectionDefault.model('App', AppSchema)
Copy link
Contributor

@nour-borgi nour-borgi Oct 25, 2023

Choose a reason for hiding this comment

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

Please add empty line at the end. Actually you need to run this: npm run lint:fix
It will prettier and lint the code

@medchedli
Copy link
Member Author

medchedli commented Nov 1, 2023

Duplicate of #1212

@medchedli medchedli closed this Nov 1, 2023
@medchedli medchedli marked this as a duplicate of #1212 Nov 1, 2023
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