Skip to content

[FIX] Missing RocketApps input types#10394

Merged
rodrigok merged 11 commits intodevelopfrom
rocketapp
Apr 16, 2018
Merged

[FIX] Missing RocketApps input types#10394
rodrigok merged 11 commits intodevelopfrom
rocketapp

Conversation

@ggazzo
Copy link
Copy Markdown
Member

@ggazzo ggazzo commented Apr 9, 2018

There are some fields we the we have not implemented yet (all fields on image are working)

image

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-10394 April 9, 2018 21:57 Inactive
@ggazzo ggazzo had a problem deploying to rocket-chat-pr-10394 April 9, 2018 21:59 Failure
@ggazzo ggazzo requested review from graywolf336 and karlprieb April 9, 2018 22:05
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-10394 April 9, 2018 22:11 Inactive

Template.appManage.onCreated(function() {

if (RocketChat.settings.cachedCollectionPrivate == null) {
Copy link
Copy Markdown
Contributor

@graywolf336 graywolf336 Apr 9, 2018

Choose a reason for hiding this comment

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

What's this for in this file? As I don't see it being used anywhere except the commented out code below on line 98..

@ggazzo ggazzo temporarily deployed to rocket-chat-pr-10394 April 10, 2018 12:57 Inactive
Copy link
Copy Markdown
Contributor

@karlprieb karlprieb left a comment

Choose a reason for hiding this comment

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

image

Select option is on the right side of the field.



Template.appManage.onCreated(function() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

???

});

Template.appManage.helpers({
// setEditorOnBlur(_id) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you remove this comment?

Copy link
Copy Markdown
Contributor

@karlprieb karlprieb left a comment

Choose a reason for hiding this comment

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

image

Select option is on the right side of the field.

@karlprieb karlprieb added the area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction label Apr 10, 2018
@karlprieb karlprieb added this to the 0.64.0 milestone Apr 10, 2018
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-10394 April 11, 2018 16:53 Inactive
@rodrigok rodrigok changed the title [NEW] Rocketapps inputs [FIX] Missing RocketApps input types Apr 16, 2018
@rodrigok rodrigok merged commit c978136 into develop Apr 16, 2018
@rodrigok rodrigok deleted the rocketapp branch April 16, 2018 15:20
@rodrigok rodrigok mentioned this pull request Apr 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants